Skip to content

T 2270/ucsfcle 405/add tool externaltaskmonitor#15

Merged
ctam merged 3 commits intoucsf-education:MOODLE_404_STABLEfrom
mirleu:T-2270/UCSFCLE_405/add-tool_externaltaskmonitor
Apr 14, 2025
Merged

T 2270/ucsfcle 405/add tool externaltaskmonitor#15
ctam merged 3 commits intoucsf-education:MOODLE_404_STABLEfrom
mirleu:T-2270/UCSFCLE_405/add-tool_externaltaskmonitor

Conversation

@mirleu
Copy link
Contributor

@mirleu mirleu commented Apr 14, 2025

Moodle4.5 Update.

Added code to explicitly call the setup() method on the parent class of our api test case. We're getting dinged for not doing so by the codesniffer in a Moodle 4.5 context, see moodlehq/moodle-cs issue 106 for details.


@mirleu mirleu marked this pull request as ready for review April 14, 2025 17:23
@mirleu mirleu requested a review from ctam April 14, 2025 17:23
Copy link
Contributor

@ctam ctam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ctam ctam merged commit 3016f30 into ucsf-education:MOODLE_404_STABLE Apr 14, 2025
4 checks passed
@mirleu mirleu deleted the T-2270/UCSFCLE_405/add-tool_externaltaskmonitor branch April 14, 2025 18:24
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.

2 participants