-
Notifications
You must be signed in to change notification settings - Fork 200
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
Add a regex configration for the signal name style rule #2204
Add a regex configration for the signal name style rule #2204
Conversation
ab3f53c
to
ab476aa
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## master #2204 +/- ##
==========================================
- Coverage 92.92% 92.92% -0.01%
==========================================
Files 359 359
Lines 26740 26768 +28
==========================================
+ Hits 24848 24874 +26
- Misses 1892 1894 +2 ☔ View full report in Codecov by Sentry. |
fec2d4e
to
56359eb
Compare
56359eb
to
c9db0ab
Compare
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.
Are you still working it it ? (as it is marked as draft)
|
Did you check the review comment so far ? |
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.
Just realised my replies were "pending"...
c9db0ab
to
980355d
Compare
Thanks for your patience Henner. I've added regex's to most of the rules listed in #2074. I'll add your feedback from this PR into those and send on through. |
980355d
to
fd79fa1
Compare
Adding regex configuration to the signal name style rule, part of issue #2074 .