Skip to content

test-requirements.txt: pin pytest to <6#512

Merged
OddBloke merged 2 commits into
canonical:masterfrom
OddBloke:pylint
Jul 29, 2020
Merged

test-requirements.txt: pin pytest to <6#512
OddBloke merged 2 commits into
canonical:masterfrom
OddBloke:pylint

Conversation

@OddBloke
Copy link
Copy Markdown
Collaborator

pylint is emitting errors with pytest 6.x which are not observed on
pytest 5.x. While that is resolved, pin to a lower pytest version.

pylint is emitting errors with pytest 6.x which are not observed on
pytest 5.x.  While that is resolved, pin to a lower pytest version.
@blackboxsw
Copy link
Copy Markdown
Collaborator

Looks like a fix landed in pytest pytest-dev/pytest#7566

@blackboxsw blackboxsw self-assigned this Jul 29, 2020
Copy link
Copy Markdown
Collaborator

@blackboxsw blackboxsw left a comment

Choose a reason for hiding this comment

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

Let's either add a comment to test-requirements.txt or a breadcrumb in the commit message about the related issue which is already fixed in 6.0.x as all we are waiting for is backport of that fix to drop this change
pytest-dev/pytest#7558

@OddBloke OddBloke merged commit db5c1c8 into canonical:master Jul 29, 2020
@OddBloke OddBloke deleted the pylint branch July 29, 2020 16:38
OddBloke added a commit to OddBloke/cloud-init that referenced this pull request Jul 31, 2020
pytest 6.0.1 fixes the issue we had with pytest 6.0.0.

This reverts commit db5c1c8.
blackboxsw pushed a commit that referenced this pull request Aug 2, 2020
pytest 6.0.1 fixes the issue we had with pytest 6.0.0.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants