-
Notifications
You must be signed in to change notification settings - Fork 499
support --ignore-words-case-sensitive option.
#3658
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
support --ignore-words-case-sensitive option.
#3658
Conversation
fujitatomoya
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@DimitriPapadopoulos what do you think about this?
- this PR keeps the current default behavior so no downstream packages are affected.
- fix the documentation aligned with current implementation in default.
- add
--ignore-words-case-sensitiveoption, to ignore all words in a case-sensitive manner. this give more flexibility for user.
|
@larsoner @peternewman could you guide me to proceed this PR? |
|
@DimitriPapadopoulos friendly ping 🙇 |
|
Hi maintainers, is there anything i can do on my side, nobody is responding at all... trying to get some attention here. |
|
@DimitriPapadopoulos could you please guide me how to get review for this PR? |
|
@larsoner @peternewman @DimitriPapadopoulos friendly ping, please. |
Signed-off-by: Tomoya Fujita <[email protected]>
89240dc to
de9a5db
Compare
|
@DimitriPapadopoulos friendly ping. |
closes #3638