Skip to content

Update GitHub Actions - #2940

Open
cthoyt wants to merge 5 commits into
OBOFoundry:masterfrom
cthoyt:patch-3
Open

Update GitHub Actions#2940
cthoyt wants to merge 5 commits into
OBOFoundry:masterfrom
cthoyt:patch-3

Conversation

@cthoyt

@cthoyt cthoyt commented Aug 19, 2026

Copy link
Copy Markdown
Collaborator

This PR does the following:

  1. Runs pinact run --update which pins each dependency in GitHub Actions to promote reproducibility
  2. Adds minor spacing to make GitHub Actions configurations more readable
  3. Update to running on modern Python 3.14

Warning

to accept this PR, the branch protection rules need to get updated, since there's no longer going to be a py310-tagged run

@cthoyt
cthoyt marked this pull request as draft August 19, 2026 13:59
@cthoyt
cthoyt marked this pull request as ready for review August 19, 2026 14:23
@cthoyt
cthoyt requested a review from jsstevenson August 19, 2026 14:27
@jsstevenson

jsstevenson commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

I think branch rule management is restricted to owners... unfortunately I don't have permissions to alter that section of the repo settings

Comment thread .github/workflows/build.yml Outdated

- name: "Install uv"
uses: "astral-sh/setup-uv@cec208311dfd045dd5311c1add060b2062131d57" # v8.0.0
uses: astral-sh/setup-uv@20cfd1bf945f4377ade1205e4dbc17946fc9a30d # v10.0.1

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why do we have that if you use pip install in the next step?

@cthoyt cthoyt Aug 31, 2026

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it doesn't appear it's used directly used. I could remove it in this PR, but I think we would want to use it to modernize some of these scripts in the near future

Comment thread .github/workflows/qa.yml Outdated
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.

3 participants