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

Explicit non-support for tox 4 #160

Merged
merged 4 commits into from
Dec 25, 2022
Merged

Explicit non-support for tox 4 #160

merged 4 commits into from
Dec 25, 2022

Conversation

ryanhiebert
Copy link
Collaborator

As mentioned in #159 , the simplest way to clean this up is to make the non-support for Tox 4 explicit, that way existing installs shouldn't break.

@ryanhiebert
Copy link
Collaborator Author

I think that I'm stuck. I've got pre-commit happy, but it's not required. The required checks are for Travis and AppVeyor, and those aren't working, presumably the setup in Travis is no longer active. I'll leave this open, but at this point there's really nothing further that I can do without help from others on the tox-dev. Additionally, apart from being willing to add this upper bound version constraint, there's really not much more I'm interested in doing with this repository. I haven't used Travis CI for ages. So I'm not even sure how much anyone else should be interested in getting this library updated, either.

@codeBKashif
Copy link

I think that I'm stuck. I've got pre-commit happy, but it's not required. The required checks are for Travis and AppVeyor, and those aren't working, presumably the setup in Travis is no longer active. I'll leave this open, but at this point there's really nothing further that I can do without help from others on the tox-dev. Additionally, apart from being willing to add this upper bound version constraint, there's really not much more I'm interested in doing with this repository. I haven't used Travis CI for ages. So I'm not even sure how much anyone else should be interested in getting this library updated, either.

@ryanhiebert With your PR, the tox-travis is working again, that start getting failing after latest release of tox==4.0. Thanks for the effort.

@gaborbernat gaborbernat merged commit e11bacf into master Dec 25, 2022
@gaborbernat gaborbernat deleted the tox4 branch December 25, 2022 17:53
@ryanhiebert
Copy link
Collaborator Author

@gaborbernat Thank you for merging! I'm pretty sure the last release was done via auto-upload in Travis. What will it take to release this version?

@gaborbernat
Copy link
Member

gaborbernat commented Dec 25, 2022 via email

@ryanhiebert
Copy link
Collaborator Author

OK, thanks. I am, so I'll figure out how to get this released. Thank you for your help getting things unblocked.

colindean added a commit to colindean/deprecated that referenced this pull request Jan 16, 2023
tantale pushed a commit to tantale/deprecated that referenced this pull request May 27, 2023
tantale pushed a commit to tantale/deprecated that referenced this pull request May 27, 2023
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

3 participants