Skip to content

chore: migrate from deptrac to structarmed - #98

Merged
samsonasik merged 2 commits into
codeigniter4:developfrom
samsonasik:migrate-to-structarmed
Sep 12, 2026
Merged

chore: migrate from deptrac to structarmed#98
samsonasik merged 2 commits into
codeigniter4:developfrom
samsonasik:migrate-to-structarmed

Conversation

@samsonasik

@samsonasik samsonasik commented Sep 11, 2026

Copy link
Copy Markdown
Member

Description

This PR migrate deptrac to structarmed, synced with shield repo. Applied Presets and rules

  • Presets: PSR4 (updated missing namespace in composer.json), CodeQuality (applied LargeNumericLiteralMustUseSeparatorRule to use separator _ on large numeric.
  • Finalize rule: MustBeFinalRule to finalize classes in tests layer, applied with single command:
vendor/bin/structarmed analyze --fix

I also fix rector deprecation notices.

Checklist:

  • Securely signed commits
  • Component(s) with PHPDoc blocks, only if necessary or adds value
  • Unit testing, with >80% coverage
  • User guide updated
  • Conforms to style guide

@samsonasik

Copy link
Copy Markdown
Member Author

Ready to review/merge 👍

@michalsn michalsn left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good - thank you!

@samsonasik

Copy link
Copy Markdown
Member Author

Thank you @michalsn for the review, let's merge 👍

@samsonasik
samsonasik merged commit c90e52d into codeigniter4:develop Sep 12, 2026
21 checks passed
@samsonasik
samsonasik deleted the migrate-to-structarmed branch September 12, 2026 10:11
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