Skip to content

Add code linting using Laravel Pint#940

Merged
AndrewKostka merged 7 commits into
mainfrom
add-code-linting
Aug 26, 2025
Merged

Add code linting using Laravel Pint#940
AndrewKostka merged 7 commits into
mainfrom
add-code-linting

Conversation

@AndrewKostka
Copy link
Copy Markdown
Contributor

Bug: T394986

Copy link
Copy Markdown
Member

@outdooracorn outdooracorn left a comment

Choose a reason for hiding this comment

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

Looking through the changes this linting applies, I'm happy with most of them. I would vote to change a few things:

  1. headsDownCamelCase instead of snake_case for function names - this should remove a lot of changes
  2. keep function's opening brace ({) on the same line as the function definition
  3. keep a space around the string concatenation operator (.) - I think it makes it look less like a full stop and more like an operator.

@AndrewKostka
Copy link
Copy Markdown
Contributor Author

Let's use .git-blame-ignore-revs to ignore the resulting squash commit.

@AndrewKostka AndrewKostka marked this pull request as ready for review August 21, 2025 11:39
@AndrewKostka
Copy link
Copy Markdown
Contributor Author

Copy link
Copy Markdown
Contributor

@rosalieper rosalieper left a comment

Choose a reason for hiding this comment

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

LGTM!

@AndrewKostka AndrewKostka merged commit 8439969 into main Aug 26, 2025
5 checks passed
@AndrewKostka AndrewKostka deleted the add-code-linting branch August 26, 2025 09:01
@deer-wmde deer-wmde mentioned this pull request Nov 7, 2025
deer-wmde pushed a commit that referenced this pull request Dec 15, 2025
* Add code linting using Laravel Pint

* Update linting ruleset

* Automatically fix linting errors
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