Skip to content

Add uv.lock to .gitignore for library projects#577

Merged
hombit merged 2 commits intomainfrom
copilot/add-uv-lock-to-gitignore
Mar 16, 2026
Merged

Add uv.lock to .gitignore for library projects#577
hombit merged 2 commits intomainfrom
copilot/add-uv-lock-to-gitignore

Conversation

Copy link
Contributor

Copilot AI commented Mar 16, 2026

Add uv.lock to .gitignore for "library" projects only. ("Command" and "notebook" packages may choose to actually have it in the repository for reproducible environments.)

One of the motivations is GSoC candidate PRs. It looks like some candidates use uv and are frustrated by uv.lock needing to be committed.

Closes #576

- Rename .gitignore to .gitignore.jinja to enable Jinja templating
- Add conditional block: uv.lock is ignored for 'library' projects, not for 'command', 'notebook', or 'custom'
- Add test to verify conditional behavior across project types

Co-authored-by: hombit <1784493+hombit@users.noreply.github.com>
Copilot AI changed the title [WIP] Add uv.lock to .gitignore Add uv.lock to .gitignore for library projects Mar 16, 2026
Copilot AI requested a review from hombit March 16, 2026 13:54
@hombit hombit requested review from drewoldag and removed request for hombit March 16, 2026 13:56
@hombit hombit marked this pull request as ready for review March 16, 2026 13:56
Copy link
Collaborator

@drewoldag drewoldag left a comment

Choose a reason for hiding this comment

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

This seems reasonable enough to me.

@hombit hombit merged commit a0589e0 into main Mar 16, 2026
8 checks passed
@hombit hombit deleted the copilot/add-uv-lock-to-gitignore branch March 16, 2026 17:27
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.

Add uv.lock to .gitignore

3 participants