Skip to content
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

Allow showing the diff when using --in-place or --check with --diff #128

Merged
merged 1 commit into from Nov 12, 2022

Conversation

BenjaminSchubert
Copy link
Contributor

It can be useful to show the difference, e.g. in CI so that users know what is or should be changed.

This adds a --diff option, that, when combined with --check or --in-place, will, in addition, show the diff to stdout.

fix #125

@coveralls
Copy link

Pull Request Test Coverage Report for Build 3422844005

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 0.0%

Totals Coverage Status
Change from base Build 2920138685: 0.0%
Covered Lines: 0
Relevant Lines: 0

💛 - Coveralls

@coveralls
Copy link

coveralls commented Nov 9, 2022

Pull Request Test Coverage Report for Build 3434976275

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 0.0%

Totals Coverage Status
Change from base Build 2920138685: 0.0%
Covered Lines: 0
Relevant Lines: 0

💛 - Coveralls

It can be useful to show the difference, e.g. in CI so that users know
what is or should be changed.

This adds a `--diff` option, that, when combined with `--check` or
`--in-place`, will, in addition, show the diff to stdout.
@weibullguy weibullguy merged commit d9a3b61 into PyCQA:master Nov 12, 2022
@BenjaminSchubert BenjaminSchubert deleted the bschubert/diff branch November 12, 2022 18:13
@weibullguy weibullguy added the S: merged Closed with work merged to repository label Dec 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S: merged Closed with work merged to repository
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow showing the diff and exit 1 if files were changed
3 participants