Skip to content

feature: Add align_multiline_comment rule to @Symfony#6875

Merged
keradus merged 1 commit intoPHP-CS-Fixer:masterfrom
yguedidi:add-align_multiline_comment-to-symfony
Apr 3, 2023
Merged

feature: Add align_multiline_comment rule to @Symfony#6875
keradus merged 1 commit intoPHP-CS-Fixer:masterfrom
yguedidi:add-align_multiline_comment-to-symfony

Conversation

@yguedidi
Copy link
Copy Markdown
Contributor

@yguedidi yguedidi commented Apr 2, 2023

After a discussion with @stof, this rule should be part of the @Symfony PHP-CS-Fixer ruleset.
Symfony code base is (almost, see symfony/symfony#49895) following it already

@yguedidi yguedidi changed the title Add align_multiline_comment rule to @Symfony feature: Add align_multiline_comment rule to @Symfony Apr 2, 2023
@yguedidi yguedidi marked this pull request as ready for review April 2, 2023 01:36
@keradus
Copy link
Copy Markdown
Member

keradus commented Apr 2, 2023

Thanks for the PR!
I'm eager to merge it when we have PR in Symfony repo merged or someone from Sf stakeholder confirms it here.

@coveralls
Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 4585902061

  • 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 93.004%

Totals Coverage Status
Change from base Build 4584756507: 0.0%
Covered Lines: 22734
Relevant Lines: 24444

💛 - Coveralls

nicolas-grekas added a commit to symfony/symfony that referenced this pull request Apr 3, 2023
This PR was merged into the 6.3 branch.

Discussion
----------

Apply align_multiline_comment PHP-CS-Fixer rule

| Q             | A
| ------------- | ---
| Branch?       | 6.3
| Bug fix?      | no
| New feature?  | no
| Deprecations? | no
| Tickets       |
| License       | MIT
| Doc PR        |

After a discussion with `@stof`, this rule should be part of the ``@Symfony`` PHP-CS-Fixer ruleset.
This PR apply the rule on the Symfony code base.
see PHP-CS-Fixer/PHP-CS-Fixer#6875 for the PR on PHP-CS-Fixer

Commits
-------

6e984f0 Apply align_multiline_comment PHP-CS-Fixer rule
@yguedidi
Copy link
Copy Markdown
Contributor Author

yguedidi commented Apr 3, 2023

@keradus PR merged on Symfony side

@keradus keradus merged commit fb7f352 into PHP-CS-Fixer:master Apr 3, 2023
@yguedidi yguedidi deleted the add-align_multiline_comment-to-symfony branch April 3, 2023 21:04
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