Fix ESLint setup #202

2022-10-06T10:31:31Z

raucao commented

2022-10-06 10:31:31 +00:00

(Migrated from github.com)

babel-eslint isn't working with node 16 anymore.

In addition to the new packages, also turn the remaining Ember errors into warnings, so we can fix them without npm test failing in the meantime.

`babel-eslint` isn't working with node 16 anymore. In addition to the new packages, also turn the remaining Ember errors into warnings, so we can fix them without `npm test` failing in the meantime.

bumi (Migrated from github.com) approved these changes 2022-10-06 11:40:16 +00:00

bumi (Migrated from github.com) left a comment

not sure about the config, I don't know much there, but looks good to me.

raucao commented

2022-10-06 12:02:14 +00:00

(Migrated from github.com)

You can easily verify the config by running npm test on your machine and checking that there are no errors and the command exits without an error code, too.

You can easily verify the config by running `npm test` on your machine and checking that there are no errors and the command exits without an error code, too.

bumi commented

2022-10-06 13:00:18 +00:00

(Migrated from github.com)

I had ran it, got no errors just "classic classes" warnings.

👍 1

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: kredits/kredits-web#202