Skip to content

Removing files from coveragerc since they have been deleted #8417

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 2 commits into from
Mar 9, 2023

Conversation

ollie-iterators
Copy link
Contributor

@ollie-iterators ollie-iterators commented Mar 9, 2023

Type of Changes

Type
πŸ› Bug fix
✨ New feature
βœ“ πŸ”¨ Refactoring
πŸ“œ Docs

Description

The TODO statement in this file says to remove the file names when the files have been deleted, so I think this is a good time to remove the files from ".coveragerc".

Refs #8405

@mbyrnepr2 mbyrnepr2 added Maintenance Discussion or action around maintaining pylint or the dev workflow Skip news πŸ”‡ This change does not require a changelog entry labels Mar 9, 2023
@codecov
Copy link

codecov bot commented Mar 9, 2023

Codecov Report

Merging #8417 (63de079) into main (d04734d) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #8417   +/-   ##
=======================================
  Coverage   95.69%   95.69%           
=======================================
  Files         175      175           
  Lines       18454    18454           
=======================================
  Hits        17659    17659           
  Misses        795      795           

Copy link
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.

Nice catch, thank you !

@Pierre-Sassoulas Pierre-Sassoulas enabled auto-merge (squash) March 9, 2023 18:52
@Pierre-Sassoulas Pierre-Sassoulas added this to the 3.0.0a6 milestone Mar 9, 2023
@Pierre-Sassoulas Pierre-Sassoulas merged commit 6789fad into pylint-dev:main Mar 9, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Maintenance Discussion or action around maintaining pylint or the dev workflow Skip news πŸ”‡ This change does not require a changelog entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants