No unused vars as errors #814
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
3 Ignored Deployments
|
|
Does this now block the local build as well? Or is there a way to ignore that locally? It would be extremely annoying to be blocked by this during development. Let's say you want to try something out. You comment out a block of code, replace it with a |
This is why linting and building should be separate. Then you can still build. But I see we're not there yet. So let's wait with merging this one until #612 is merged first. |
webpro
force-pushed
the
feat/no-unused-vars-as-errors
branch
from
6ca1bf2
to
f9dc61c
Compare
webpro
force-pushed
the
feat/no-unused-vars-as-errors
branch
from
f9dc61c
to
513a422
Compare
🦋 Changeset detectedLatest commit: 38cda2b The changes in this PR will be included in the next version bump. This PR includes changesets to release 0 packagesWhen changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
There was a problem hiding this comment.
When we merge this file like this. Will all subsequent changesets not trigger a "missing" changeset, and inadvertently result in missing changeset entries when we release?
There was a problem hiding this comment.
I don't think so. Let's keep an eye on it.
As I think unused variables are not just warnings, but errors.