Skip to content

docs: add guidance on pinning a commit hash for security purposes#21

Merged
samuelmatos-s1 merged 2 commits intomainfrom
guidance-patch
Mar 30, 2026
Merged

docs: add guidance on pinning a commit hash for security purposes#21
samuelmatos-s1 merged 2 commits intomainfrom
guidance-patch

Conversation

@samuelmatos-s1
Copy link
Copy Markdown
Contributor

@samuelmatos-s1 samuelmatos-s1 commented Mar 30, 2026

Adds guidance on pinning commit hash when running uvx.

@samuelmatos-s1 samuelmatos-s1 marked this pull request as ready for review March 30, 2026 22:54
@github-actions

This comment has been minimized.

@samuelmatos-s1 samuelmatos-s1 merged commit 1582c09 into main Mar 30, 2026
14 checks passed
@samuelmatos-s1 samuelmatos-s1 deleted the guidance-patch branch March 30, 2026 22:58
@github-actions
Copy link
Copy Markdown

AI Analysis

Here are some suggestions to consider:

  • Appreciate the new security guidance around pinning commits—great addition! One small tweak in README.md: the sentence "from the our releases" has an extra "the". Dropping it to read "from our releases" will keep the note polished and easy to follow.

These suggestions are generated by gpt-5-codex as an experimental capability. It may make mistakes. You do not need to follow all these suggestions.

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.

1 participant