Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Prepare for Moodle 4.3 and PHP 8.2. #542

Merged
merged 2 commits into from
Apr 4, 2024

Conversation

lucaboesch
Copy link
Contributor

I have a working branch here that fixes #530.

@mchurchward
Copy link
Contributor

Thanks @lucaboesch . It looks like there are new errors? Probably due to the merges in MOODLE_401_STABLE.

@lucaboesch lucaboesch force-pushed the MOODLE_403_STABLE branch 3 times, most recently from f2b373d to e5c0ccb Compare February 24, 2024 12:14
@lucaboesch
Copy link
Contributor Author

Checks are passing again, now, @mchurchward. A branch MOODLE_403_STABLE in your repository would be highly appreciated, I think.

@mchurchward
Copy link
Contributor

@lucaboesch I'm not sure these Behat changes are required. The test pass for me as is.

@lucaboesch lucaboesch force-pushed the MOODLE_403_STABLE branch 2 times, most recently from 0334a66 to 12ad081 Compare March 14, 2024 23:39
@lucaboesch
Copy link
Contributor Author

Yes, you are right, @mchurchward.
I've reset the Behat test to the original.

@lucaboesch lucaboesch changed the base branch from MOODLE_401_STABLE to MOODLE_400_STABLE March 14, 2024 23:45
@lucaboesch lucaboesch changed the base branch from MOODLE_400_STABLE to MOODLE_401_STABLE March 14, 2024 23:46
@mchurchward
Copy link
Contributor

Hi Luca. Sorry, can you rebase again?

@mchurchward mchurchward mentioned this pull request Apr 3, 2024
@mchurchward
Copy link
Contributor

@lucaboesch I'm not sure what happened, but there are 14 commits in this PR and it doesn't seem like they all belong to this PR.

@lucaboesch lucaboesch force-pushed the MOODLE_403_STABLE branch 6 times, most recently from 37fa63d to cb65739 Compare April 3, 2024 19:25
@lucaboesch
Copy link
Contributor Author

That should look better now, @mchurchward .

@mchurchward mchurchward merged commit b06947c into PoetOS:MOODLE_401_STABLE Apr 4, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Creation of dynamic properties is deprecated
2 participants