Skip to content

Comments

Add Larastan and Pint to CI#20

Merged
luisdalmolin merged 16 commits intomainfrom
feature/add-larastan
May 7, 2025
Merged

Add Larastan and Pint to CI#20
luisdalmolin merged 16 commits intomainfrom
feature/add-larastan

Conversation

@nathanheffley
Copy link
Member

This pull request installs Larastan (set to level 0 with a baseline file to avoid style changes in this PR) and adds both Pint and Larastan checks to the CI action. This will resolve #15

@nathanheffley nathanheffley requested a review from adammparker May 5, 2025 19:25
@nathanheffley nathanheffley self-assigned this May 5, 2025
@nathanheffley
Copy link
Member Author

I set Larastan to Level 0 and added a baseline file to ignore some violations (mostly uses of static in non-final classes) because there are quite a few more even on level 1 and I wanted to keep this PR focused as much on pure CI as possible. I did fix the styles in three files though so that the style linter passes cleanly.

@nathanheffley nathanheffley force-pushed the feature/add-larastan branch from 1c4bb17 to fd89759 Compare May 7, 2025 14:51
@nathanheffley nathanheffley requested a review from luisdalmolin May 7, 2025 16:06
@luisdalmolin luisdalmolin merged commit 4639821 into main May 7, 2025
6 checks passed
@luisdalmolin luisdalmolin deleted the feature/add-larastan branch May 7, 2025 17:24
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.

Add CI for Linting/Static Analysis

3 participants