Skip to content

Use --mode=production on the linter so console.logs cause failure again#206

Merged
JackWilb merged 3 commits intomainfrom
fix-linter
Nov 17, 2021
Merged

Use --mode=production on the linter so console.logs cause failure again#206
JackWilb merged 3 commits intomainfrom
fix-linter

Conversation

@JackWilb
Copy link
Copy Markdown
Member

Closes #201

The environment for the linter was somehow being set to development when linting. Explicitly setting it to production solves that problem and now all code must be linted to production standards to be commited and merged.

@netlify
Copy link
Copy Markdown

netlify bot commented Nov 17, 2021

✔️ Deploy Preview for next-multinet-client ready!

🔨 Explore the source changes: 58504ad

🔍 Inspect the deploy log: https://app.netlify.com/sites/next-multinet-client/deploys/61953bcc49b6390008c1a46b

😎 Browse the preview: https://deploy-preview-206--next-multinet-client.netlify.app/

@JackWilb JackWilb merged commit b89f07c into main Nov 17, 2021
@JackWilb JackWilb deleted the fix-linter branch November 17, 2021 17:32
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.

eslint warnings not triggering errors on build

1 participant