csharpier icon indicating copy to clipboard operation
csharpier copied to clipboard

Look into implementing caching to speed up formatting.

Open belav opened this issue 3 years ago • 3 comments

Prettier recently added caching to determine if formatting a file can be skipped. https://prettier.io/docs/en/cli.html#--cache

Look into what they are doing and determine if a similar feature can be added to csharpier.

belav avatar Jun 20 '22 16:06 belav

Is that really necessary? I feel like csharpier is already bloody fast. :stuck_out_tongue:

shocklateboy92 avatar Jul 13 '22 06:07 shocklateboy92

But what if we can make it super bloody fast? It is ~22 seconds on our work main repo (13,000 files or so) without the syntax checker. If caching can drop that down to like 5 seconds that would be awesome. I'd do a quick POC first to see if it makes sense to go through with it.

belav avatar Jul 13 '22 21:07 belav

My initial POC using last modified date - the 13,000 file repo can be formatted in 3 seconds if caching is used and no files have changed. 10x improvement!

belav avatar Jul 18 '22 19:07 belav