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.

Sign up for GitHub

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

Jump to bottom

CI failure on main: documentation check #8762

Closed
jacobtylerwalls opened this issue Jun 12, 2023 · 1 comment · Fixed by #8772
Closed

CI failure on main: documentation check #8762

jacobtylerwalls opened this issue Jun 12, 2023 · 1 comment · Fixed by #8772
Labels
Beta-Blocker 🙅🩸 If this issue is not fixed before the beta release, our blood pressure increase too much Bug 🪲 Needs PR This issue is accepted, sufficiently specified and now needs an implementation
Milestone
3.0.0a7

Comments

@jacobtylerwalls
Copy link
Member

Bug description

The last two commits on main fail this documentation check.

Since it began when merging #8757, I double-checked that #8757 wasn't to blame by reverting it on a PR on my fork; didn't help. (Makes sense, test failure didn't come up when merging the PR.)

Can't reproduce locally.

Configuration

No response

Command used

CI main branch

Pylint output

See failing action

Expected behavior

CI passes (it does locally)

Pylint version

CI main branch

OS / Environment

No response

Additional dependencies

No response
@jacobtylerwalls jacobtylerwalls added Needs triage 📥 Just created, needs acknowledgment, triage, and proper labelling Bug 🪲 Beta-Blocker 🙅🩸 If this issue is not fixed before the beta release, our blood pressure increase too much labels Jun 12, 2023
@Pierre-Sassoulas Pierre-Sassoulas added Needs PR This issue is accepted, sufficiently specified and now needs an implementation and removed Needs triage 📥 Just created, needs acknowledgment, triage, and proper labelling labels Jun 12, 2023
@jacobtylerwalls jacobtylerwalls added this to the 3.0.0a7 milestone Jun 12, 2023
@Pierre-Sassoulas Pierre-Sassoulas mentioned this issue Jun 12, 2023
Closed
@jacobtylerwalls
Copy link
Member Author

jacobtylerwalls commented Jun 13, 2023 •
edited
Loading

Was able to reproduce it locally with tox instead of pytest directly.

@jacobtylerwalls jacobtylerwalls mentioned this issue Jun 13, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Beta-Blocker 🙅🩸 If this issue is not fixed before the beta release, our blood pressure increase too much Bug 🪲 Needs PR This issue is accepted, sufficiently specified and now needs an implementation
Projects
None yet
Milestone
3.0.0a7
Development

Successfully merging a pull request may close this issue.

Add __init__.py files to message test dirs pylint-dev/pylint
2 participants
@Pierre-Sassoulas @jacobtylerwalls