Skip to content

Comments

used uv to install python#144

Merged
wmalgadey merged 5 commits intoerwindouna:mainfrom
wmalgadey:remove-poetry-from-linting
Feb 18, 2026
Merged

used uv to install python#144
wmalgadey merged 5 commits intoerwindouna:mainfrom
wmalgadey:remove-poetry-from-linting

Conversation

@wmalgadey
Copy link
Collaborator

Proposed Changes

final move to remove poetry from linting

Related Issues

relates to #139

@wmalgadey wmalgadey self-assigned this Feb 18, 2026
@wmalgadey wmalgadey marked this pull request as ready for review February 18, 2026 22:01
Copilot AI review requested due to automatic review settings February 18, 2026 22:01
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the GitHub Actions linting workflow to rely on uv for Python installation, aligning with the goal of removing Poetry from the linting pipeline (related to workflow stability in #139).

Changes:

  • Replace actions/setup-python with uv python install ${{ env.DEFAULT_PYTHON }} across all linting jobs.
  • Remove the actions/setup-python uv cache configuration from the workflow.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@wmalgadey wmalgadey added the github_actions Pull requests that update GitHub Actions code label Feb 18, 2026
@wmalgadey wmalgadey added the ci Work that improve the CI/CD label Feb 18, 2026
@wmalgadey wmalgadey merged commit 9a2df65 into erwindouna:main Feb 18, 2026
11 checks passed
@wmalgadey wmalgadey deleted the remove-poetry-from-linting branch February 18, 2026 22:26
@github-actions github-actions bot locked and limited conversation to collaborators Feb 20, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

ci Work that improve the CI/CD github_actions Pull requests that update GitHub Actions code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant