Issue #1071: chore: remove the unused setup-poetry composite action - #1306
Merged
bjagg merged 1 commit intoSep 24, 2026
Merged
Conversation
…osite action The last Poetry in the build path was .github/actions/setup-poetry, and no workflow references it: every service builds with uv (LIF-Initiative#1067, LIF-Initiative#1068), and a repo-wide search for "setup-poetry" under .github/ finds only the action's own file. Deleting it leaves uv as the single Python build toolchain. The action also pip-installed earthmover. Nothing runs that install today, since the action is never called, so removing it changes no build. Remaining "poetry" matches in the repo are stock .gitignore template comments and the English word in schema/sample data, none of them build tooling. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Description of Change
Problem. #1071 asks us to standardize on uv and retire Poetry from the build path. The only Poetry left in the build path is the
setup-poetrycomposite action.Solution. Delete
.github/actions/setup-poetry/action.yml. No workflow calls it. Every service builds with uv (#1067, #1068), and a search forsetup-poetryunder.github/finds only the action's own file. After this, uv is the single Python build toolchain.Side effects. None expected. The action also ran
pip install earthmover, but since nothing calls the action, that install never runs today.Not changed, deliberately:
.gitignore/orchestrators/dagster/lif-orchestrator/.gitignore: the# poetrylines are stock Python.gitignoretemplate comments, not tooling.Evidence.genredescriptions).docs/operations/proposals/cf-ci-improvements.mditem 6 still describes the action. That document is an outside review recording what it found, so it is left as a record rather than edited.How to test.
git grep -n "setup-poetry" -- .githubreturns nothing on this branch, and the PR-time CI plus every deploy workflow run unchanged because none of them referenced the action.Related Issues
Closes #1071
Refs #1074
Type of Change
Project Area(s) Affected
.github/actions/(CI composite actions)Checklist
pr-ci.Testing
setup-poetryreferences in.github/🤖 Generated with Claude Code