Proposal: new hook to detect LFS attribute inconsistencies
Nobody has claimed this yet.
Assessment
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Newbie friendliness
- 35/100
Research direction
Start by reviewing the proposed check-lfs-attributes implementation at the linked branch, then compare it with the repository's existing hook patterns. Verify both reported cases: tracked files that should be LFS pointers and LFS pointers whose attributes were removed; done means the hook detects each inconsistency reliably.
Written by the indexing model from the issue text.
Description
There are two common errors when working with LFS:
- Keeping files as general objects after setting LFS attributes (LFS produces a warnings like "Encountered 1 file(s) that should have been pointers, but weren't")
- Keeping LFS pointers after removing the attributes.
Proposed implementation of a check-lfs-attributes hook: https://github.com/klimkin/pre-commit-hooks/tree/feature/check-lfs-attributes
- Dominant language
- Python
- Stars
- 6.7k
- Forks
- 800
- PR merge metrics
- No merged PRs in 30d
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from pre-commit/pre-commit-hooks
-
Immutable Releases Open
Difficulty 2/5 1-3 hours Newbie friendliness 45/100
pre-commit/pre-commit-hooks#1218 · 4 comments · 2 reactions ·
-
Difficulty 4/5 3-5 days Newbie friendliness 35/100
pre-commit/pre-commit-hooks#780 · 5 comments · 2 reactions ·
-
Difficulty 3/5 1-2 days Newbie friendliness 48/100
pre-commit/pre-commit-hooks#749 · 1 reaction ·
-
enhancement good-first-issue
Difficulty 3/5 1-2 days Newbie friendliness 55/100
pre-commit/pre-commit-hooks#612 · 13 comments ·
-
Difficulty 5/5 Over a week Newbie friendliness 25/100
pre-commit/pre-commit-hooks#339 · 18 comments · 2 reactions ·
All issues in pre-commit/pre-commit-hooks
Similar issues
-
documentation help wanted
Difficulty 2/5 1-3 hours Newbie friendliness 90/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 90/100
simonw/sqlite-utils#872 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100