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

CI: update python versions for lint.yml #332

Merged
merged 2 commits into from
Dec 29, 2024

Conversation

ev-br
Copy link
Contributor

@ev-br ev-br commented Dec 28, 2024

Python 3.8 is not available on ubuntu-latest images anymore.

Python 3.8 is not available on ubuntu-latest images anymore.
@ev-br ev-br enabled auto-merge December 28, 2024 15:40
@ev-br
Copy link
Contributor Author

ev-br commented Dec 28, 2024

I've no idea where the stalling "buld: (3.8)" and "build: (3.9)" jobs are coming from. Clearing the actions cache does not help. Unless it's going to somehow resolve itself, the "lint" CI job in in this repo is broken. Maybe not that big of a loss anywhay.

@rgommers
Copy link
Member

Not sure it's the root cause, but setup-python@v1 is very old. Maybe update actions to latest version?

@rgommers
Copy link
Member

Linting on two separate Python versions isn't useful I think, so also better to remove the matrix-ing over Python versions while you're at it.

@rgommers
Copy link
Member

And Ubuntu-provided packages shouldn't matter? The Python comes from setup-python, which supports 3.8/3.9 just fine.

@ev-br
Copy link
Contributor Author

ev-br commented Dec 29, 2024

Good idea, thanks. Updated the action versions, removed the matrix. Stalling builds are still there.

@rgommers rgommers disabled auto-merge December 29, 2024 21:56
@rgommers rgommers merged commit f7a74a6 into data-apis:master Dec 29, 2024
3 checks passed
@rgommers
Copy link
Member

I can't find the problem so quickly. Let's see if it resolves itself now that the job is updated.

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