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

Fix GHA 2.7 support #214

Merged
merged 8 commits into from
Sep 29, 2023
Merged

Fix GHA 2.7 support #214

merged 8 commits into from
Sep 29, 2023

Conversation

gotcha
Copy link
Member

@gotcha gotcha commented Jul 27, 2023

No description provided.

@gotcha gotcha force-pushed the fix_gha branch 3 times, most recently from 8b7ec66 to 4597548 Compare July 27, 2023 10:16
@gotcha gotcha marked this pull request as draft July 27, 2023 10:18
@gotcha gotcha force-pushed the fix_gha branch 11 times, most recently from 49d68eb to 541d558 Compare July 27, 2023 14:51
@pbauer
Copy link
Member

pbauer commented Jul 31, 2023

What about the approach used here: actions/setup-python#672 (comment)

@gotcha
Copy link
Member Author

gotcha commented Aug 2, 2023

@pbauer 30f517f does the job. But caching on GHA is broken.

That could be merged if you like.

@pbauer
Copy link
Member

pbauer commented Sep 18, 2023

Cool, this seems to work now. Thanks for that! Using the image plone/test-with-tox for the 2.7 tests is fine with me.

@pbauer
Copy link
Member

pbauer commented Sep 19, 2023

@gotcha is that ready to be reviewed/merged?

@gotcha
Copy link
Member Author

gotcha commented Sep 20, 2023

I just wanted to use the same approach on another repo to see if it still needed to be improved before merging. Should be this week.

@pbauer
Copy link
Member

pbauer commented Sep 29, 2023

@gotcha ready to merge now?

@pbauer pbauer marked this pull request as ready for review September 29, 2023 15:08
@pbauer pbauer merged commit e9505b5 into main Sep 29, 2023
@pbauer pbauer deleted the fix_gha branch September 29, 2023 15:08
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