-
Notifications
You must be signed in to change notification settings - Fork 10
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
Version 10 #207
Version 10 #207
Conversation
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #207 +/- ##
===========================================
Coverage 100.00% 100.00%
Complexity 150 150
===========================================
Files 8 8
Lines 611 669 +58
===========================================
+ Hits 611 669 +58 ☔ View full report in Codecov by Sentry. |
@Khartir @valentin-schaeidt-mayflower could you check the Attributes rules in your respective project and report back if this fits your needs? |
Minor issue: sorting the attributes alphabetically includes a leading More importantly, I just had this happen: #[Bar()]
- // TODO do something
public function foo(): void |
Thanks, I opened slevomat/coding-standard#1497. |
dfea223
to
079f688
Compare
@Khartir Could you check again? :) |
It still does weird stuff: //phpcs:disable SlevomatCodingStandard.PHP.ForbiddenClasses
-#[Bar()] /** foo */
class TestCase But I think we can merge this and either fix it upstream or disable offending sniffs downstream. |
Quality Gate passedKudos, no new issues were introduced! 0 New issues |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me.
No description provided.