Skip to content

Add __init__.py files to message test dirs#8772

Merged
Pierre-Sassoulas merged 1 commit intomainfrom
fix-doc-check
Jun 13, 2023
Merged

Add __init__.py files to message test dirs#8772
Pierre-Sassoulas merged 1 commit intomainfrom
fix-doc-check

Conversation

@jacobtylerwalls
Copy link
Copy Markdown
Member

Type of Changes

Type
🐛 Bug fix

Description

Closes #8762

Other doc checks that have multiple files under good/ or bad/ have an __init__.py file. The specific way tox was launching the tests was causing astroid to expect an __init__.py file, and these two didn't have them.

@jacobtylerwalls jacobtylerwalls added tests Skip news 🔇 This change does not require a changelog entry labels Jun 13, 2023
@jacobtylerwalls jacobtylerwalls changed the title Add __init__ files to message test dirs Add __init__.py files to message test dirs Jun 13, 2023
@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 13, 2023

Codecov Report

Merging #8772 (514ede6) into main (7eaac57) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #8772   +/-   ##
=======================================
  Coverage   95.83%   95.83%           
=======================================
  Files         173      173           
  Lines       18437    18437           
=======================================
  Hits        17669    17669           
  Misses        768      768           

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.

Nice catch, I was working in it with less success 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Skip news 🔇 This change does not require a changelog entry tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CI failure on main: documentation check

2 participants