Skip to content

Release/502 - #44

Merged
TamaroWalter merged 7 commits into
mainfrom
release/502
Apr 7, 2026
Merged

Release/502#44
TamaroWalter merged 7 commits into
mainfrom
release/502

Conversation

@TamaroWalter

Copy link
Copy Markdown
Member

🔀 Purpose of this PR:

  • Fixes a bug
  • Updates for a new Moodle version
  • Adds a new feature of functionality
  • Improves or enhances existing features
  • Refactoring: restructures code for better performance or maintainability
  • Testing: add missing or improve existing tests
  • Miscellaneous: code cleaning (without functional changes), documentation, configuration, ...

📝 Description:

This is the pull request for Moodle 5.2.

Changes:

  • resolve warnings/errors in php mess detector, js and mustache linter
  • codecleaning
  • use datasets in templates/js for data instead of placing data into css ids.

📋 Checklist

Please confirm the following (check all that apply):

  • I have phpunit and/or behat tests that cover my changes or additions.
  • Code passes the code checker without errors and warnings.
  • Code passes the moodle-ci/cd pipeline on all supported Moodle versions or the ones the plugin supports.
  • Code does not have var_dump() or var_export or any other debugging statements (or commented out code) that
    should not appear on the productive branch.
  • Code only uses language strings instead of hard-coded strings.
  • If there are changes in javascript: I build new .min files with the grunt amd command.
  • If it is a Moodle update PR: I read the release notes, updated the version.php and the CHANGES.md.
    I ran all tests thoroughly checking for errors. I checked if bootstrap had any changes/deprecations that require
    changes in the plugins UI.

@TamaroWalter
TamaroWalter merged commit 0df0991 into main Apr 7, 2026
48 checks passed
@TamaroWalter
TamaroWalter deleted the release/502 branch April 7, 2026 21:28
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.

1 participant