Skip to content

Conversation

@pgbaumann
Copy link

@pgbaumann pgbaumann commented Nov 23, 2025

Fixes #3663

When using the -w flag 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:

  • Added changes_made list to track fixes made to a file during write mode
  • Print list of changes after "FIXED: filename"
  • Moved color formatting code into a function to avoid code duplication
  • Added a test

@pgbaumann pgbaumann changed the title Feat/show detailed changes when using w flag Feat/add inline summaries in --write-changes mode Nov 23, 2025
@pgbaumann pgbaumann changed the title Feat/add inline summaries in --write-changes mode Add inline summaries in --write-changes mode Nov 23, 2025
@DimitriPapadopoulos
Copy link
Collaborator

Ah, the test is an excellent idea, thank you for that.

@larsoner
Copy link
Member

@DimitriPapadopoulos feel free to merge if you're happy

@DimitriPapadopoulos
Copy link
Collaborator

@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.

@pgbaumann
Copy link
Author

@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.

@pgbaumann pgbaumann force-pushed the feat/show-detailed-changes-when-using-w-flag branch from f4e0cc9 to d0f15fa Compare November 26, 2025 20:23
@pgbaumann
Copy link
Author

@DimitriPapadopoulos I have updated the PR by resolving conflicts with #3832.
Please take a look when you have a chance.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add inline summaries in --write-changes mode

3 participants