ci: streamline workflows #7739
Merged
ci: streamline workflows #7739
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
psychedelicious
requested review from
lstein,
blessedcoolant,
hipsterusername and
ebr
as code owners
github-actions
bot
added
CI-CD
psychedelicious
force-pushed
the
psyche/ci/streamline-workflows
branch
from
01babb0 to
a33dda4
Compare
|
I've simulated failures of various types and everything is working as it should. We'll need an override merge for this because it removes 4x workflows from the python test matrix. |
|
Oh, there is one weird thing. I couldn't make So, it still uses the standard setup-python action. This can be squash-merged, there was a bit of fumbling to get the pieces fitting together. |
|
Main benefit to this change is faster CI for python tests - |
ebr
approved these changes
Mar 7, 2025
psychedelicious
force-pushed
the
psyche/ci/streamline-workflows
branch
from
ce7ee40 to
007d7df
Compare
hipsterusername
approved these changes
Mar 7, 2025
psychedelicious
force-pushed
the
psyche/ci/streamline-workflows
branch
from
007d7df to
c309131
Compare
We only have CPU runners, so these tests are not doing anything useful.
psychedelicious
force-pushed
the
psyche/ci/streamline-workflows
branch
2 times, most recently
from
d01e2d8 to
dedacf1
Compare
|
I've cleaned up the commits in this PR, bumped versions of ubuntu for |
psychedelicious
force-pushed
the
psyche/ci/streamline-workflows
branch
from
3ca8bb8 to
ba26da4
Compare
Need to bump both CI and pyproject.toml at the same time
Lots of squashed experimentation heh: ci: manually specify python version in tests ci: whoops typo in ruff cmds ci: specify python versions for uv python install ci: install python verbosely ci: try forcing python preference? ci: try forcing python preference a different way? ci: try in a venv? ci: it works, but try without venv ci: oh maybe we need --preview? ci: poking it with a stick ci: it works, add summary to pytest output ci: fix pytest output experiment: simulate test failure Revert "experiment: simulate test failure" This reverts commit b99ca51. ci: just use default pytest output cI: attempt again to use uv to install python cI: attempt again again to use uv to install python Revert "cI: attempt again again to use uv to install python" This reverts commit 3cba861. Revert "cI: attempt again to use uv to install python" This reverts commit b30f227.
ci: use uv in typegen-checks to generate types experiment: simulate typegen-checks failure Revert "experiment: simulate typegen-checks failure" This reverts commit f53c687.
psychedelicious
force-pushed
the
psyche/ci/streamline-workflows
branch
from
ba26da4 to
ec0435b
Compare
1 task
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
uvfor everything.Related Issues / Discussions
n/a
QA Instructions
It it fits, I sits.
Merge Plan
n/a
Checklist
What's Newcopy (if doing a release after this PR)