Skip to content

Use StructArmed on QA - #49

Merged
alexmerlin merged 3 commits into
dotkernel:2.0from
samsonasik:use-structarmed-on-qa
Oct 28, 2026
Merged

alexmerlin merged 3 commits into
dotkernel:2.0from
samsonasik:use-structarmed-on-qa

Conversation

@samsonasik

Copy link
Copy Markdown
Contributor

This PR to make use of StructArmed.

This PR start apply StructArmed with initialize using PSR4 and CODEQUALITY preset and already fixes 31 violations.

On PSR4 preset, it help fixes class namespaces on tests.

The CODEQUALITY apply make use of static in anonymous function when possible, applied with single command:

vendor/bin/structarmed analyze --fix

I also enable ruleset above so it show more ability of layering setup to define which layers each layer is allowed to depend on.

Signed-off-by: Abdul Malik Ikhsan <samsonasik@gmail.com>
Signed-off-by: Abdul Malik Ikhsan <samsonasik@gmail.com>
@samsonasik
samsonasik force-pushed the use-structarmed-on-qa branch from 0ae17a0 to 51b32b4 Compare October 27, 2026 23:46
@samsonasik

Copy link
Copy Markdown
Contributor Author

@arhimede @alexmerlin ready for review 👍

Comment thread composer.json Outdated
Signed-off-by: Abdul Malik Ikhsan <samsonasik@gmail.com>

@alexmerlin alexmerlin 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.

Thanks @samsonasik

@alexmerlin
alexmerlin merged commit 9935bf9 into dotkernel:2.0 Oct 28, 2026
11 of 12 checks passed
@samsonasik
samsonasik deleted the use-structarmed-on-qa branch October 28, 2026 05:02
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