Skip to content

NEW option --git-only - Implements #1897 #3300 - #3676

Open
mdeweerd wants to merge 1 commit into
codespell-project:mainfrom
mdeweerd:feat/git_filter
Open

NEW option --git-only - Implements #1897 #3300#3676
mdeweerd wants to merge 1 commit into
codespell-project:mainfrom
mdeweerd:feat/git_filter

Conversation

@mdeweerd

@mdeweerdmdeweerd commented Mar 25, 2025•
edited
Loading

Copy link
Copy Markdown
Contributor

NEW option --git-only - Implements #1897#3300

codespell --git-only only checks files under git control.

It has been useful to me several times over the past two weeks already.
Installed using

pip install --upgrade git+https://github.com/mdeweerd/codespell.git@feat/git_filter

@mdeweerd
mdeweerd marked this pull request as ready for review March 25, 2025 11:58
@mdeweerd

mdeweerd commented Mar 25, 2025•
edited
Loading

Copy link
Copy Markdown
ContributorAuthor

Open for review of the functionality, feedback on how you want to handle the security notices (ignore, do it differently).

Regarding tests; I accept contributions to my fork to update this PR. I've added the tests.

@mdeweerdmdeweerd mentioned this pull request Mar 25, 2025
Open
DimitriPapadopoulos
DimitriPapadopoulos reviewed Mar 26, 2025
View reviewed changes
Comment threadcodespell_lib/_codespell.py Outdated
DimitriPapadopoulos
DimitriPapadopoulos reviewed Mar 26, 2025
View reviewed changes
Comment threadcodespell_lib/_codespell.py
@mdeweerd

mdeweerd commented Mar 26, 2025

Copy link
Copy Markdown
ContributorAuthor

@DimitriPapadopoulos - Thank you for the feedback.

@mdeweerd
mdeweerdforce-pushed the feat/git_filter branch 4 times, most recently from 70fe03c to 05751d1CompareMarch 27, 2025 21:59
@mdeweerdmdeweerd changed the title Fix #1897 #3300 Add git_filterFix #1897 #3300 Add --git-onlyMar 27, 2025
@mdeweerdmdeweerd changed the title Fix #1897 #3300 Add --git-onlyNEW option --git-only - Implements #1897 #3300Apr 9, 2025
@shasheeneshasheene mentioned this pull request Jun 12, 2025
Closed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@DimitriPapadopoulosDimitriPapadopoulosDimitriPapadopoulos left review comments
@larsonerlarsonerAwaiting requested review from larsonerlarsoner is a code owner
@peternewmanpeternewmanAwaiting requested review from peternewmanpeternewman is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mdeweerd@DimitriPapadopoulos