Skip to content

Lock file maintenance (cakephp-5.x) - #34

Open
renovate[bot] wants to merge 1 commit into
cakephp-5.xfrom
renovate/cakephp-5.x-lock-file-maintenance
Open

Lock file maintenance (cakephp-5.x)#34
renovate[bot] wants to merge 1 commit into
cakephp-5.xfrom
renovate/cakephp-5.x-lock-file-maintenance

Conversation

@renovate

@renovate renovate Bot commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

This PR contains the following updates:

Update Change
lockFileMaintenance All locks refreshed

🔧 This Pull Request updates lock files to use the latest dependency versions.


Configuration

📅 Schedule: (in timezone Europe/Zurich)

  • Branch creation
    • "before 4am on monday"
  • Automerge
    • At any time (no schedule defined)

🚦 Automerge: Enabled.

Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.

👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.


  • If you want to rebase/retry this PR, check this box

This PR was generated by Mend Renovate. View the repository job log.

@renovate
renovate Bot force-pushed the renovate/cakephp-5.x-lock-file-maintenance branch from d70419b to e7d7003 Compare August 12, 2026 00:40
@Neluxx

Neluxx commented Aug 12, 2026

Copy link
Copy Markdown
Member

@ravage84 Here we have a case where the CI check for the lowest possible dependencies fails.
We could now identify which dependency is causing the failure and bump it to a version where all checks pass again.

The question that arises here is: Is it worth the effort, and are we responsible for ensuring that even the lowest possible dependencies work, or can we ignore this?

@ravage84

Copy link
Copy Markdown
Member

@ravage84 Here we have a case where the CI check for the lowest possible dependencies fails. We could now identify which dependency is causing the failure and bump it to a version where all checks pass again.

The question that arises here is: Is it worth the effort, and are we responsible for ensuring that even the lowest possible dependencies work, or can we ignore this?

Once we start ignoring build errors, the whole setup becomes questionable.

The error comes from Slevomat's PHP CS Standard:

> phpcs
ERROR: Property "newlinesCountAfterDeclare" does not exist on sniff SlevomatCodingStandard.TypeHints.DeclareStrictTypes.
ERROR: Property "newlinesCountBetweenOpenTagAndDeclare" does not exist on sniff SlevomatCodingStandard.TypeHints.DeclareStrictTypes.
ERROR: Property "enableStaticTypeHint" does not exist on sniff SlevomatCodingStandard.TypeHints.PropertyTypeHint.

Run "phpcs --help" for usage information

Script phpcs handling the cs-check event returned with error code 3
Script @cs-check was called via check
Error: Process completed with exit code 3.

Apparently, even with PHP 8.4 with the lowest dependencies, some CS config properties which are referenced somewhere aren't available.

[...]
    - Downgrading cakephp/cakephp-codesniffer (5.3.1 => 5.0.0)
[...]
    - Downgrading slevomat/coding-standard (8.31.1 => 8.4.0)
[...]

With the lowest dependencies in the CI matrix, the lowest dependencies will be fetched through composer update --prefer-lowest..

prefer-lowest:

Prefer lowest versions of dependencies. Useful for testing minimal versions of requirements, generally used with --prefer-stable. [...]

That likely means our lower bound requirement for cakephp/cakephp-codesniffer is too low.

Likely, the CakePHP Codesniffer ruleset references these config properties mentioned above., at some point.

E.g. newlinesCountAfterDeclare was introduced in commit slevomat/coding-standard@52cca43 and released in 8.31.1 but CakePHP's CodeSniffer lowest bound requirement is 8.23 of Slevomat's., even in the newest version.
CakePHP CodeSniffer had it removed in commit cakephp/cakephp-codesniffer@4728b69, again .

Solution: Bump the requirement of cakephp/cakephp-codesniffer to the version after the removal of all config properties mentioned above. Version 5.1.1 still contains references to enableStaticTypeHint.

FYI, @orca-services

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