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

Unpin tox #1093

Merged
merged 5 commits into from
Feb 2, 2023
Merged

Unpin tox #1093

merged 5 commits into from
Feb 2, 2023

Conversation

hynek
Copy link
Member

@hynek hynek commented Jan 28, 2023

It works locally for me.

@hynek
Copy link
Member Author

hynek commented Jan 28, 2023

no it doesn't

@hynek hynek closed this Jan 28, 2023
@webknjaz
Copy link
Member

Wait, if it works locally but not in GHA, maybe it's the plugin's fault? I think I've seen a corresponding plugin for tox4 being separate from the one for tox3.

@hynek
Copy link
Member Author

hynek commented Jan 28, 2023

no, I ran tox -e mypy locally but the broken thing is the test suite. I've isolated & opened a bug: tox-dev/tox#2904

@hynek hynek deleted the unpin-tox branch January 28, 2023 12:49
@webknjaz
Copy link
Member

Ah, I see. But FTR there's https://github.com/tox-dev/tox-gh too.

@hynek
Copy link
Member Author

hynek commented Jan 28, 2023

What does that to do with anything? Our problem is that the package pins for Mypy are from PyPI instead of from the local package. I don't see how the plugins play into that.

@webknjaz
Copy link
Member

It's unrelated to the mypy problem but is related to the tox4 migration, IIUC.

@hynek hynek restored the unpin-tox branch January 29, 2023 12:19
@hynek hynek reopened this Jan 29, 2023
@hynek
Copy link
Member Author

hynek commented Jan 29, 2023

It's unrelated to the mypy problem but is related to the tox4 migration, IIUC.

According to https://pypi.org/project/tox-gh-actions/, tox-gh-actions>=3 works with tox 3 just fine.

I would switch if there's any upsides, but I can't see any rn?


Anyways, tox has fixed the bug 🎉 and we're waiting for a release now.

@webknjaz
Copy link
Member

webknjaz commented Jan 29, 2023

According to https://pypi.org/project/tox-gh-actions/, tox-gh-actions>=3 works with tox 3 just fine.

Looks like it's ~= 2.0 though.

@webknjaz
Copy link
Member

FWIW here's some notes regarding the differences tox-dev/tox-gh#7 (comment)

@hynek
Copy link
Member Author

hynek commented Feb 1, 2023

what the flying fuck is going on now… it works perfectly fine on my machine!?

@webknjaz
Copy link
Member

webknjaz commented Feb 1, 2023

Looks related to the recent Hatch-scm integration. At least one of the logs failed on int('dev2').
Also, I saw a resource warning so there might be an unclosed file descriptor somewhere.

@hynek
Copy link
Member Author

hynek commented Feb 1, 2023

but but but… it used to work! otherwise i wouldn’t have merged it! :(

I’m currently in the process of setting up a new computer, once I’m fully functional I’ll look closer sigh

@webknjaz
Copy link
Member

webknjaz commented Feb 1, 2023

FWIW I checked this branch locally and it does work. Maybe somebody needs to stick https://github.com/marketplace/actions/debugging-with-tmate there and inspect the worker over SSH.

Oh, and I just remembered the most typical setuptools-scm mistake — the contributors usually have a full Git clone locally but CIs tend to do shallow clones. The fix is usually to set the fetch depth to 0 or run git unshallow.

Though, maybe this is not the only problem here, I haven't had time to dig deeper.

@hynek
Copy link
Member Author

hynek commented Feb 2, 2023

0c9b54c #genius

@hynek hynek merged commit 95d8a76 into main Feb 2, 2023
@hynek hynek deleted the unpin-tox branch February 2, 2023 05:43
@hynek
Copy link
Member Author

hynek commented Feb 2, 2023

FTR on tox-gh: it seems like it has a very different way of working than tax-gh-actions. I tries using it with structlog and 3.11 = py311 means literally "only run py311" while tox-gh-actions essentially says -f py311. For attrs it doesn't matter for now, but that's a very significant different:

CleanShot 2023-02-02 at 08 16 50@2x

(using screenshot b/c logs get lost over time)

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