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

bug: CurlyBracesPositionFixer - fix for open brace not preceded by space and followed by a comment #6776

Merged
merged 3 commits into from
Feb 8, 2023
Merged

Conversation

kubawerlos
Copy link
Contributor

@kubawerlos kubawerlos commented Feb 7, 2023

Fixes #6775

Take a look at the changes in the first commit - the new test is failing not because of the changes not being as in the test, but because of broken PHP syntax.

@kubawerlos kubawerlos marked this pull request as ready for review February 7, 2023 17:04
@coveralls
Copy link

coveralls commented Feb 7, 2023

Pull Request Test Coverage Report for Build 4116614885

  • 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.0003%) to 92.95%

Totals Coverage Status
Change from base Build 4093635880: 0.0003%
Covered Lines: 22703
Relevant Lines: 24425

💛 - Coveralls

@keradus keradus merged commit 655f3c4 into PHP-CS-Fixer:master Feb 8, 2023
@kubawerlos kubawerlos deleted the fix_CurlyBracesPositionFixer branch February 8, 2023 19:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Error: Missing block "end"
3 participants