Add new moodle.PHPUnit.TestCasesAbstract sniff to check that testcase classes are declared abstract.
Add new moodle.PHPUnit.TestClassesFinal sniff to check that test classes are declared final.
Add new moodle.Commenting.Package sniff to replace those present in moodle-local_moodlecheck.
Add new moodle.Commenting.Category sniffs to replace those present in moodle-local_moodlecheck.
New phpcs.xml.dist to enforce the coding style to follow by ´moodle-cs´ itself. Basically, PSR12 ruled. CI verified from now on.
Changed
Modified own CI scripts, updating various GH actions.
Reallocated own tests structure, to better organise them and fixture files.
Update composer dependencies to current versions, notably PHP_CodeSniffer (3.9.0) and PHPCompatibility (e5cd2e24).
As part of the move to be PSR12 compliant, all the methods used for testing have been converted, without deprecation, to camel case (setStandard(), setSniff(), ...).
ACTION REQUIRED: Any clone/fork using moodle-cs and having own tests will need to adapt them to the new method names.
Fixed
The moodle.Files.MoodleInternal sniff no longer treats Attributes as side-effects.
Solved a problem with the moodle.Commenting.Package sniff when running against arbitrary (not Moodle based) directories.
[v3.4.0] - 2024-03-07
Added
Add new moodle.PHPUnit.TestCasesAbstract sniff to check that testcase classes are declared abstract.
Add new moodle.PHPUnit.TestClassesFinal sniff to check that test classes are declared final.
Add new moodle.Commenting.Package sniff to replace those present in moodle-local_moodlecheck.
Add new moodle.Commenting.Category sniffs to replace those present in moodle-local_moodlecheck.
New phpcs.xml.dist to enforce the coding style to follow by moodle-cs itself. Basically, PSR12 ruled. CI verified from now on.
Changed
Modified own CI scripts, updating various GH actions.
Reallocated own tests structure, to better organise them and fixture files.
Update composer dependencies to current versions, notably PHP_CodeSniffer (3.9.0) and PHPCompatibility (e5cd2e24).
As part of the move to be PSR12 compliant, all the methods used for testing have been converted, without deprecation, to camel case (setStandard(), setSniff(), ...).
ACTION REQUIRED: Any clone/fork using moodle-cs and having own tests will need to adapt them to the new method names.
Fixed
The moodle.Files.MoodleInternal sniff no longer treats Attributes as side-effects.
Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.
Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it
- `@dependabot merge` will merge this PR after your CI passes on it
- `@dependabot squash and merge` will squash and merge this PR after your CI passes on it
- `@dependabot cancel merge` will cancel a previously requested merge and block automerging
- `@dependabot reopen` will reopen this PR if it is closed
- `@dependabot close` will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
- `@dependabot show ignore conditions` will show all of the ignore conditions of the specified dependency
- `@dependabot ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
- `@dependabot ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
- `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
Bumps moodlehq/moodle-cs from 3.3.15 to 3.4.1.
Release notes
Sourced from moodlehq/moodle-cs's releases.
Changelog
Sourced from moodlehq/moodle-cs's changelog.
Commits
3966e9f
Prepare the v3.4.1 released7be382
Merge pull request #118 from stronk7/prevent_check_when_no_commponentf06fa34
Disable package checks whenever missing a target componentf71d22b
Prepare the v3.4.0 release1cdcfa0
Rename a too-much generic provider and test namede914c0
Merge pull request #117 from stronk7/code_cleanup_psr12313e171
Add phpcs execution to GHA, plus change log updates970108b
Change the testing API (MoodleCSBaseTestCase) to camelCase1c12b33
Make the whole codebase compliant with our configurationa50b294
Add the phpcs.xml dist file and some small adjustmentsDependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase
.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR: - `@dependabot rebase` will rebase this PR - `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it - `@dependabot merge` will merge this PR after your CI passes on it - `@dependabot squash and merge` will squash and merge this PR after your CI passes on it - `@dependabot cancel merge` will cancel a previously requested merge and block automerging - `@dependabot reopen` will reopen this PR if it is closed - `@dependabot close` will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually - `@dependabot show