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

Ignore single next line #301

Merged
merged 2 commits into from
Mar 2, 2023
Merged

Conversation

RKushnir
Copy link
Contributor

Allows to ignore the coverage for the following line of code, by putting a new type of comment # coveralls-ignore-next-line.

Ignoring a single line is a common use case, which currently can be done by wrapping the code between the "start" and "stop" comments. If we can achieve the same with just one comment, this produces less visual noise and leaves more space for the useful code.

Resolves #219

Allows to ignore the coverage for the following line of code,
by putting a new type of comment `# coveralls-ignore-next-line`.

Ignoring a single line is a common use case, which currently can be done by wrapping the code
between the "start" and "stop" comments. If we can achieve the same with just one comment,
this produces less visual noise and leaves more space for the useful code.

Resolves parroty#219
@coveralls
Copy link

Coverage Status

Coverage: 88.082% (+0.3%) from 87.828% when pulling 8c9b42b on RKushnir:ignore-single-next-line into 6890fdc on parroty:master.

@coveralls
Copy link

coveralls commented Mar 2, 2023

Coverage Status

Coverage: 87.896% (+0.07%) from 87.828% when pulling 8c9b42b on RKushnir:ignore-single-next-line into 6890fdc on parroty:master.

@parroty parroty merged commit dc10f97 into parroty:master Mar 2, 2023
@parroty
Copy link
Owner

parroty commented Mar 2, 2023

Thanks!

@RKushnir
Copy link
Contributor Author

RKushnir commented Mar 2, 2023

Wow, thanks for accepting it! 🎉

@RKushnir RKushnir deleted the ignore-single-next-line branch March 2, 2023 13:42
@kianmeng
Copy link
Contributor

@RKushnir Thanks for this PR, appreciate it!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feature request: ignore one line
4 participants