Skip to content

qa: install architecture guard package boundwize/structarmed - #843

Merged
romm merged 4 commits into
CuyZ:masterfrom
samsonasik:init-structarmed
Sep 21, 2026
Merged

romm merged 4 commits into
CuyZ:masterfrom
samsonasik:init-structarmed

Conversation

@samsonasik

Copy link
Copy Markdown
Contributor

@romm here initial use of StructArmed with PSR4 Preset on QA.

This apply to Source layer that PSR4 preset scans: qa and src directory to ease for gradual updates.

Per I write at #841 (comment), there is nothing to showoff to just PSR4 preset on qa and src, I can apply on tests directory, but there are many multiple classes in single file which can be discussed later whether they are on purpose.

@samsonasik samsonasik mentioned this pull request Sep 21, 2026
@samsonasik

Copy link
Copy Markdown
Contributor Author

@romm ready to merge 👍

@samsonasik

Copy link
Copy Markdown
Contributor Author

@romm if you need to include CodeQuality so can showoff here, just let me know ;)

@romm

romm commented Sep 21, 2026

Copy link
Copy Markdown
Member

As stated in #841 (comment), could you update the composer check and composer fix commands as well?

@samsonasik

Copy link
Copy Markdown
Contributor Author

@romm I've applied 2 new commits:

  • registered to composer check and composer fix script at commit 2c0ee0c
  • register and apply code quality preset at 29790ff . Tested with run:
composer fix

Just let me know if you need to revert the CodeQuality preset showoff :)

@romm romm changed the title Initial use of StructArmed with PSR4 Preset qa: install architecture guard package boundwize/structarmed Sep 21, 2026
@romm
romm merged commit 7d80d4e into CuyZ:master Sep 21, 2026
16 checks passed
@samsonasik
samsonasik deleted the init-structarmed branch September 21, 2026 15:43
@romm

romm commented Sep 21, 2026

Copy link
Copy Markdown
Member

Good, thank you 😊

Now we can work on the rulesets!

@samsonasik

Copy link
Copy Markdown
Contributor Author

@romm sure, I created new PR for ruleset:

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