Skip to content

Conversation

@JanTvrdik
Copy link
Member

Summary

  • Replaces slevomat/coding-standard with shipmonk/coding-standard: ^0.2.0
  • Simplifies phpcs.xml.dist by using ShipMonkCodingStandard ruleset instead of manually specifying all rules

Test plan

  • CI passes

@JanTvrdik JanTvrdik force-pushed the migrate-to-shipmonk-coding-standard branch 2 times, most recently from d13d156 to 28792fa Compare November 26, 2025 12:43
@JanTvrdik
Copy link
Member Author

depends on #60

- Replace slevomat/coding-standard with shipmonk/coding-standard ^0.2.0
- Simplify phpcs.xml.dist to use ShipMonkCodingStandard ruleset
- Remove conflicting DisallowTrailingCommaInDeclaration rule
- Apply code style fixes (multi-line signatures, short nullable format)
@JanTvrdik JanTvrdik force-pushed the migrate-to-shipmonk-coding-standard branch from 28792fa to c965b4b Compare November 27, 2025 10:53
@JanTvrdik JanTvrdik requested a review from janedbal November 27, 2025 10:54
@janedbal janedbal merged commit a08dfdb into master Nov 27, 2025
10 checks passed
@janedbal janedbal deleted the migrate-to-shipmonk-coding-standard branch November 27, 2025 11:10
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.

3 participants