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.

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

T252577: Tell about older tox versions no longer working #125

Merged
merged 3 commits into from
Feb 19, 2024

Conversation

gandie
Copy link
Member

@gandie gandie commented Feb 19, 2024

Mention nasty version struggle in Caveats chapter. There are even more traps behind the initial error shown, so people should know and just take the right steps.

@gandie gandie reopened this Feb 19, 2024
README.md Outdated
@@ -310,4 +310,9 @@ containing broken `External Method`s are recorded and played back. The same
holds for `Z SQL Method`s which have `class_name` and `class_file` set to a no
longer existing extension.

Since `22.2.5` the latest version of `tox` is required in order to build the
Copy link
Collaborator

Choose a reason for hiding this comment

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

Maybe a formulation like "a recent enough version" may be better here?

Copy link
Member Author

Choose a reason for hiding this comment

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

Well "recent enough" was too vague for my taste. Ran through changelog and tested, version 3.26.0 is the first one working again, which corresponds nicely with the bugfixes mentioned as we use isolated_build nowadays:

tox v3.26.0 Bugfixes

Copy link
Collaborator

@viktordick viktordick left a comment

Choose a reason for hiding this comment

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

LGTM

@gandie gandie merged commit 11e8a86 into master Feb 19, 2024
3 checks passed
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.

None yet

2 participants