Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use a Github Action to check for code style #2429

Merged
merged 2 commits into from
Sep 6, 2021
Merged

Use a Github Action to check for code style #2429

merged 2 commits into from
Sep 6, 2021

Conversation

lucasmichot
Copy link
Collaborator

@lucasmichot lucasmichot commented Aug 22, 2021

This PR provides a Github Action as a replacement for StyleCI.

StyleCI has been introduced 6 years ago when Github Actions were not a thing yet, and has done its job well.
Since then Github Action has been introduced and provide a greater flexibility and we can use only PHP-CS-Fixer.

This way we can

  • manage rules and ruleset in a single .php-cs-fixer.dist.php file
  • check code against many rulesets ( PSR2 + PSR12 + ... )
  • no need to wait a PHP-CS-Fixer rule to be updated or added on StyleCI side
    etc...

Cf. code-check : https://github.com/briannesbitt/Carbon/pull/2429/checks?check_run_id=3400609992


After this PR is merged, this repo should be disabled in StyleCI admin

@kylekatarnls kylekatarnls added this to the 2.54.0 milestone Sep 6, 2021
@kylekatarnls kylekatarnls merged commit 5e14b87 into briannesbitt:master Sep 6, 2021
@lucasmichot lucasmichot deleted the feat/use-ga-php-cs-fixer branch May 13, 2022 09:40
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