Skip to content

Refactor _is_type_checking_import of unused-import#5410

Merged
DanielNoord merged 2 commits intopylint-dev:mainfrom
DanielNoord:import
Nov 27, 2021
Merged

Refactor _is_type_checking_import of unused-import#5410
DanielNoord merged 2 commits intopylint-dev:mainfrom
DanielNoord:import

Conversation

@DanielNoord
Copy link
Copy Markdown
Collaborator

  • Add yourself to CONTRIBUTORS if you are a new contributor.
  • Add a ChangeLog entry describing what your PR does.
  • If it's a new feature, or an important bug fix, add a What's New entry in
    doc/whatsnew/<current release.rst>.
  • Write a good description on what the PR does.

Type of Changes

Type
🔨 Refactoring
📜 Docs

Description

#5316 (comment) and adds a changelog for that PR.

@DanielNoord DanielNoord added the Maintenance Discussion or action around maintaining pylint or the dev workflow label Nov 27, 2021
@coveralls
Copy link
Copy Markdown

coveralls commented Nov 27, 2021

Pull Request Test Coverage Report for Build 1511355072

  • 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.002%) to 93.504%

Totals Coverage Status
Change from base Build 1506998000: -0.002%
Covered Lines: 13991
Relevant Lines: 14963

💛 - Coveralls

Comment thread ChangeLog Outdated
Co-authored-by: Marc Mueller <30130371+cdce8p@users.noreply.github.com>
Copy link
Copy Markdown
Member

@cdce8p cdce8p left a comment

Choose a reason for hiding this comment

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

Looks good. Thanks @DanielNoord 🐬
Github is having some issue at the moment, so I think it's fine even if the CodeQL job fails.

@DanielNoord DanielNoord merged commit 4b8881c into pylint-dev:main Nov 27, 2021
@DanielNoord DanielNoord deleted the import branch November 27, 2021 23:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Maintenance Discussion or action around maintaining pylint or the dev workflow

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants