phetsims / aqua

Automatic QUality Assurance
MIT License
2 stars 4 forks source link

QuickServer should always lint without cache #186

Closed zepumph closed 1 year ago

zepumph commented 1 year ago

We found that Typescript 5.1 didn't show lint rules on quick server, but did create lint rules, let's try disabling the cache each time we lint.

zepumph commented 1 year ago

@samreid feels like this shouldn't occur every time. Let's see about detecting the cases where we need to manually break the cache:

For right now though we will see if it takes TOO LONG as is. Closing, but will reopen if it proves challeneging.

zepumph commented 1 year ago

Quick server is taking 5.5 minutes per lint. Let's not just call this case closed.

zepumph commented 1 year ago

We will revert for now and reopen if we have lots of caching in the future.