-
Notifications
You must be signed in to change notification settings - Fork 581
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
RSTcheckBear: Allow ignoring roles and directives #1723
Comments
c.f. #1718 for a similar enhancement |
Instead of adding RSTcheckBear here. I can fix this issues in #4211 itself. |
No. This is an enhancement for RSTcheckBear. |
Ohk so nothing has to be done with .coafile. I will have to go through RSTcheckBear. |
I understood what needs to be done. But for checking if everything works fine. I will have to add rstcheckBear to the .coafile of coala-bears repository. |
To test your changes, you need to add tests to |
Missing coverage 9-15. |
Can someone please help me with this. I am unable to get 100% code coverage. And not sure if changes are right or not. |
@mridubhatnagar it seems you are not working on the issue any more, and @umeshksingla said he has an urgent need for this feature. If you don't mind he would take this issue over :) @umeshksingla please accept the invitation, then I'll assign the issue to you :) |
Unassigned due to inactivity. |
Thanks @mridubhatnagar for helping. Closes coala#1723
Thanks @mridubhatnagar for helping. Closes coala#1723
Thanks @mridubhatnagar for helping. Closes coala#1723
Thanks @mridubhatnagar for helping. Closes coala#1723
Thanks @mridubhatnagar for helping. Closes coala#1723
rstcheck has cmdline args for ignoring directives and ignoring roles.
These need to be exposed via bear settings. This is blocking a proper fix for coala/coala#4211 (comment)
The text was updated successfully, but these errors were encountered: