-
-
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
Make hints optional #52
Conversation
Made the use of hints optional, given that, for some rules, the title is descriptive enough.
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #52 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 5 5
Lines 148 149 +1
=========================================
+ Hits 148 149 +1
☔ View full report in Codecov by Sentry. |
6978053
to
b754703
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.
Hi @El-Virus,
Thank you for your contribution. I always appreciate the help and people involvement in my passion projects.
I left you a couple of comments to further improve your contribution. I also took the liberty and added a test, as writing a test in an unfamiliar setup can be hard for first-time contributors to a new project.
Cheers!
Joe
b754703
to
b00dc82
Compare
Note: I've 0 experience with codecov & have no idea why it's failing its tests. |
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.
No worries. I improved the tests and slightly simplified the current implementation to facilitate your use case with less conditional code blocks. I believe this is ready to be shipped. Thanks @El-Virus excellent contribution 👍
Made the use of hints optional, given that, for some rules, the title is descriptive enough.