Skip to content

Make missing-yield/raises-doc respect no-docstring-rgx option#7421

Merged
DanielNoord merged 1 commit intopylint-dev:mainfrom
DanielNoord:yield-raise
Sep 6, 2022
Merged

Make missing-yield/raises-doc respect no-docstring-rgx option#7421
DanielNoord merged 1 commit intopylint-dev:mainfrom
DanielNoord:yield-raise

Conversation

@DanielNoord
Copy link
Copy Markdown
Collaborator

@DanielNoord DanielNoord commented Sep 6, 2022

  • Write a good description on what the PR does.
  • Create a news fragment with towncrier create <IssueNumber>.<type> which will be
    included in the changelog. <type> can be one of: new_check, removed_check, extension,
    false_positive, false_negative, bugfix, other, internal. If necessary you can write
    details or offer examples on how the new change is supposed to work.
  • If you used multiple emails or multiple names when contributing, add your mails
    and preferred name in script/.contributors_aliases.json

Type of Changes

Type
🐛 Bug fix

Description

Closes #4743

@DanielNoord DanielNoord added Bug 🪲 Configuration Related to configuration Needs backport Needs to be cherry-picked on the current patch version by a pylint's maintainer labels Sep 6, 2022
@DanielNoord DanielNoord added this to the 2.15.1 milestone Sep 6, 2022
@coveralls
Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 2999482043

  • 6 of 6 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.002%) to 95.334%

Totals Coverage Status
Change from base Build 2997493502: 0.002%
Covered Lines: 17001
Relevant Lines: 17833

💛 - Coveralls

Copy link
Copy Markdown
Member

@Pierre-Sassoulas Pierre-Sassoulas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👌

@Pierre-Sassoulas
Copy link
Copy Markdown
Member

The fail in documentation is unrelated to the pull request.

@DanielNoord DanielNoord merged commit 63dbc60 into pylint-dev:main Sep 6, 2022
@DanielNoord DanielNoord deleted the yield-raise branch September 6, 2022 10:53
@Pierre-Sassoulas Pierre-Sassoulas added Backported and removed Needs backport Needs to be cherry-picked on the current patch version by a pylint's maintainer labels Sep 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

False positive missing-return-doc when a function is not supposed to be documented

3 participants