moodlehq / moodle-local_codechecker

A Moodle local plugin providing a simple web UI to run the MoodleCS coding style checks with PHP_CodeSniffer.
63 stars 72 forks source link

Bump moodlehq/moodle-cs from 3.3.14 to 3.3.15 #230

Closed dependabot[bot] closed 6 months ago

dependabot[bot] commented 6 months ago

Bumps moodlehq/moodle-cs from 3.3.14 to 3.3.15.

Release notes

Sourced from moodlehq/moodle-cs's releases.

v3.3.15

Added

  • Function declaration lines are verified to have correct whitespace separation (1 space or 0 spaces) between all their parts.

Changed

  • Modified own CI scripts, bumping them from NodeJS 16 to NodeJS 20 actions.

Fixed

  • PHP enum uses are not considered side-effects by some sniffs.
  • PHPDoc blocks are allowed before PHP attributes, that are immediately followed by the PHP structure.
  • The PHPUnit dataProvider sniff now reports the problematic lines properly.
  • The valid variable names sniff now detects properly escaped (\$) contents within double quoted strings and they aren't considered - for validation - anymore.
  • Avoided PHP error when checking empty PHP files.

Full changelog

Changelog

Sourced from moodlehq/moodle-cs's changelog.

[v3.3.15] - 2024-02-15

Added

  • Function declaration lines are verified to have correct whitespace separation (1 space or 0 spaces) between all their parts.

Changed

  • Modified own CI scripts, bumping them from NodeJS 16 to NodeJS 20 actions.

Fixed

  • PHP enum uses are not considered side-effects by some sniffs.
  • PHPDoc blocks are allowed before PHP attributes, that are immediately followed by the PHP structure.
  • The PHPUnit dataProvider sniff now reports the problematic lines properly.
  • The valid variable names sniff now detects properly escaped (\$) contents within double quoted strings and they aren't considered - for validation - anymore.
  • Avoided PHP error when checking empty PHP files.
Commits
  • 3a95fd8 Prepare the v3.3.15 release
  • 65d1834 Merge pull request #104 from stronk7/function_declaration_spacing
  • 23ca26f Cover all the new function declaration sniffs with own test
  • 18aff12 Check for correct whitespace in method declarations
  • 52e5f08 Merge pull request #100 from stronk7/boilerplate_sniff_and_empty_files
  • 3629cb2 Avoid PHP error with BoilerplateComment and empty files
  • 4f7f6d6 Merge pull request #105 from stronk7/better_detect_variables_in_strings
  • 807358d Discard non-variables within double quoted strings
  • 42ab278 Merge pull request #101 from stronk7/fix_provider_reported_lines
  • a6ef3c0 Fix reported lines in the TestCaseProviderSniff
  • Additional commits viewable in compare view


Dependabot compatibility score

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)
stronk7 commented 6 months ago

Thanks @dependabot !