Skip to content
This repository has been archived by the owner on Mar 31, 2021. It is now read-only.

Updates eslint config and lints everything #492

Merged
merged 8 commits into from
Nov 29, 2020

Conversation

madsnedergaard
Copy link
Member

Description

Updates the linting setup to make it more similar to main repo. Also lints all files and fixes errors + warnings.

This was done in preparation for working on the models :)

@madsnedergaard madsnedergaard marked this pull request as draft November 29, 2020 15:16
@madsnedergaard madsnedergaard marked this pull request as ready for review November 29, 2020 18:58
},
rules: {
'no-console': 'off',
'import/no-unresolved': 'off',
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure why I'm getting errors from this rule 🤔

But since it's only for the playground, I guess it might be okay to disable it here for now?

Screenshot 2020-11-29 at 15 52 50

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hm, that a bit weird. Did you install the node_modules in the playground folder?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Haha of course, missed that 😅

@madsnedergaard madsnedergaard merged commit fd11ba9 into master Nov 29, 2020
@madsnedergaard madsnedergaard deleted the mn/update-linting-setup branch November 29, 2020 20:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants