-
Notifications
You must be signed in to change notification settings - Fork 506
Add inline summaries in --write-changes mode #3836
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
Add inline summaries in --write-changes mode #3836
Conversation
|
Ah, the test is an excellent idea, thank you for that. |
|
@DimitriPapadopoulos feel free to merge if you're happy |
|
@pgbaumann I'm afraid #3832 refactored the code you've modified — mostly moving part of a function into its own function. Do you think you can find time to rebase this PR? Sorry about the annoyance. |
Hi @DimitriPapadopoulos. Sure, I will have a look next weekend. |
f4e0cc9 to
d0f15fa
Compare
for more information, see https://pre-commit.ci
|
@DimitriPapadopoulos I have updated the PR by resolving conflicts with #3832. |
|
Just a friendly ping. I've resolved the conflicts with #3832, so this should be ready for review whenever you have time. Thanks! |
|
Thank you @pgbaumann. |
Fixes #3663
When using the
-wflag to automatically fix spelling errors, codespell now displays a list of changes made below the "FIXED: filename" message, similar to the output in non-write mode.Before:
FIXED: foo.h
After:
FIXED: foo.h
foo.h:63: Creats ==> Creates
Changes:
changes_madelist to track fixes made to a file during write mode