Skip to content

Enable running code cleanup locally only on files changed since the specified branch #1173

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

Merged
merged 1 commit into from
Sep 6, 2022

Conversation

bkoelman
Copy link
Member

@bkoelman bkoelman commented Jul 31, 2022

This PR adds an optional parameter to cleanupcode.ps1, which allows formatting only changed files since a git branch or commit hash. And it now includes staged and unstaged files.

Example usage:

C:\Source\Repos\JsonApiDotNetCore [cleanupcode-diff ≡]> ./cleanupcode.ps1 master

Fixes #1112 (well, the first bullet of #1112 (comment); the second one can be addressed by importing the severities layer locally).

QUALITY CHECKLIST

@codecov
Copy link

codecov bot commented Jul 31, 2022

Codecov Report

Merging #1173 (770b848) into master (0185311) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1173   +/-   ##
=======================================
  Coverage   92.58%   92.58%           
=======================================
  Files         241      241           
  Lines        7700     7700           
=======================================
  Hits         7129     7129           
  Misses        571      571           

Help us with your feedback. Take ten seconds to tell us how you rate us.

@bkoelman bkoelman requested a review from maurei July 31, 2022 12:42
@maurei
Copy link
Member

maurei commented Sep 6, 2022

THANK you for this one. Long overdue!

@maurei maurei merged commit 206ca38 into master Sep 6, 2022
@maurei maurei deleted the cleanupcode-diff branch September 6, 2022 14:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

Optimize the build process
2 participants