Skip to content

CI: update python versions for lint.yml#332

Merged
rgommers merged 2 commits into
data-apis:masterfrom
ev-br:ci-lint-py-versions
Dec 29, 2024
Merged

CI: update python versions for lint.yml#332
rgommers merged 2 commits into
data-apis:masterfrom
ev-br:ci-lint-py-versions

Conversation

@ev-br

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

Copy link
Copy Markdown
Member

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

ev-br commented Dec 28, 2024

Copy link
Copy Markdown
Member Author

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
Copy Markdown
Member

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

@rgommers

Copy link
Copy Markdown
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
Copy Markdown
Member

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

@ev-br

ev-br commented Dec 29, 2024

Copy link
Copy Markdown
Member Author

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
@rgommers

Copy link
Copy Markdown
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