Skip to content

Fix code cleanup #1107

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 4 commits into from
Nov 18, 2021
Merged

Fix code cleanup #1107

merged 4 commits into from
Nov 18, 2021

Conversation

bart-degreed
Copy link
Contributor

@bart-degreed bart-degreed commented Nov 9, 2021

Fixes #1104.

Related to sethreno/ReGitLint#13 and sethreno/ReGitLint#14.

QUALITY CHECKLIST

  • Changes implemented in code
  • Complies with our contributing guidelines
  • N/A: Adapted tests
  • N/A: Documentation updated
  • N/A: Created issue to update Templates: {ISSUE_NUMBER}

@codecov
Copy link

codecov bot commented Nov 10, 2021

Codecov Report

Merging #1107 (3834c2e) into master (b7930de) will not change coverage.
The diff coverage is n/a.

❗ Current head 3834c2e differs from pull request most recent head d9d870a. Consider uploading reports for the commit d9d870a to get more accurate results
Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1107   +/-   ##
=======================================
  Coverage   88.60%   88.60%           
=======================================
  Files         255      255           
  Lines        7160     7160           
=======================================
  Hits         6344     6344           
  Misses        816      816           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b7930de...d9d870a. Read the comment docs.

@bart-degreed bart-degreed requested a review from maurei November 10, 2021 09:28
@maurei maurei merged commit 8a6bfac into master Nov 18, 2021
@maurei maurei deleted the fix-code-cleanup branch November 18, 2021 12:49
bart-degreed pushed a commit that referenced this pull request Dec 3, 2021
* Disable GlobalPerProduct setting layer in cleanupcode/inspectcode

* Temporary code change to verify cleanupcode still fails in cibuild

* Revert "Temporary code change to verify cleanupcode still fails in cibuild"

This reverts commit 037b58f.

* Adapted to latest version of ReGitLint
bart-degreed pushed a commit that referenced this pull request Dec 3, 2021
* Disable GlobalPerProduct setting layer in cleanupcode/inspectcode

* Temporary code change to verify cleanupcode still fails in cibuild

* Revert "Temporary code change to verify cleanupcode still fails in cibuild"

This reverts commit 037b58f.

* Adapted to latest version of ReGitLint
bart-degreed pushed a commit that referenced this pull request Dec 3, 2021
* Disable GlobalPerProduct setting layer in cleanupcode/inspectcode

* Temporary code change to verify cleanupcode still fails in cibuild

* Revert "Temporary code change to verify cleanupcode still fails in cibuild"

This reverts commit 037b58f.

* Adapted to latest version of ReGitLint
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Different dotnet tool run jb cleanupcode results locally and on appveyor
2 participants