Skip to content

Comments

fix: remove non-deterministic uv flags from docs workflow#44

Merged
pmalarme merged 2 commits intofeature/add-documentation-and-copilot-instructionsfrom
copilot/sub-pr-18-again
Feb 19, 2026
Merged

fix: remove non-deterministic uv flags from docs workflow#44
pmalarme merged 2 commits intofeature/add-documentation-and-copilot-instructionsfrom
copilot/sub-pr-18-again

Conversation

Copy link
Contributor

Copilot AI commented Feb 19, 2026

The docs CI workflow was passing -U --prerelease=if-necessary-or-explicit to uv sync, bypassing uv.lock pinning and allowing pre-release packages — making every docs build non-reproducible and a potential supply-chain risk.

Changes

  • .github/workflows/python-docs.yml: Strip -U --prerelease=if-necessary-or-explicit from extra-args; retain only --all-packages
# Before
extra-args: "--all-packages -U --prerelease=if-necessary-or-explicit"

# After
extra-args: "--all-packages"

Testing

  • Not run (why?)
  • uv run poe check
  • Other: YAML-only change; no code logic affected

Checklist

  • Linked issue or task reference
  • Added/updated tests where relevant
  • Updated docs/README if needed
  • No secrets or sensitive data added
  • Considered backward compatibility and deployment impact

Additional context


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: pmalarme <686568+pmalarme@users.noreply.github.com>
Copilot AI changed the title [WIP] Update documentation and Copilot instructions based on feedback fix: remove non-deterministic uv flags from docs workflow Feb 19, 2026
Copilot AI requested a review from pmalarme February 19, 2026 14:55
@pmalarme pmalarme marked this pull request as ready for review February 19, 2026 15:11
@pmalarme pmalarme merged commit 350d968 into feature/add-documentation-and-copilot-instructions Feb 19, 2026
@pmalarme pmalarme deleted the copilot/sub-pr-18-again branch February 19, 2026 15:11
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