search

terrazzoapp / terrazzo

Use DTCG tokens JSON to generate code for web, mobile, native apps, and more
https://terrazzo.app
MIT License
147 stars 25 forks source link

Try out --changed #245

Closed drwpow closed 2 months ago

drwpow commented 5 months ago

Changes

Tries out lint ratcheting with --changed. It’s a neat idea, turning CI from erring on all failures, to only failing on regressions. The reason you’d want to catch regressions is the default settings often discourage tightening lint rules because whoever changes the lint rule has to change the entire codebase (even sometimes requiring code refactoring).

But with --changed, lint rules can be enforced from a certain point onward, and catch new regressions while letting you slowly chip away at the existing issues over time. It’s the best of both worlds—it encourages increasingly-tightening your linting, without punishing you for existing violations in a working codebase.

How to Review

changeset-bot[bot] commented 5 months ago

⚠️ No Changeset found

Latest commit: c1f066d78ade72a7dd2afac093e44ecac0cf6281

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

cloudflare-pages[bot] commented 5 months ago

Deploying cobalt-ui with  Cloudflare Pages  Cloudflare Pages

Latest commit: c1f066d
Status: ✅  Deploy successful!
Preview URL: https://7190a6e6.cobalt-ui.pages.dev
Branch Preview URL: https://biome-changed.cobalt-ui.pages.dev

View logs