Skip to content

allow checking formatting with cache#1830

Open
sd-st wants to merge 1 commit intobelav:mainfrom
sd-st:faster-check
Open

allow checking formatting with cache#1830
sd-st wants to merge 1 commit intobelav:mainfrom
sd-st:faster-check

Conversation

@sd-st
Copy link

@sd-st sd-st commented Mar 10, 2026

Description

I run csharpier check as part of my lint flow, which means I run it a lot. On one of my repositories it takes 200ms to run format and 8000ms to run check. This PR allows for check to use the cache (although it can be disabled, just like for format)

Related Issue

Checklist

  • My code follows the project's code style
    • always var
    • follow existing naming conventions
    • always this.
    • no pointless comments
  • I will not force push after a code review of my PR has started
  • I have added tests that cover my changes

@sd-st
Copy link
Author

sd-st commented Mar 10, 2026

I think there's probably nothing to test here; it doesn't look like there were tests for the previous behaviour (and no tests for the CLI arguments in general as far as I can tell)

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant