Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Upgrade: eslint and other deps #92

Merged
merged 4 commits into from
Apr 8, 2020
Merged

Upgrade: eslint and other deps #92

merged 4 commits into from
Apr 8, 2020

Conversation

aladdin-add
Copy link
Contributor

No description provided.

@@ -11,7 +11,7 @@

const rule = require('../../../lib/rules/no-deprecated-report-api');
const RuleTester = require('eslint').RuleTester;
const ERROR = [{ message: 'Use the new-style context.report() API.', type: 'CallExpression' }];
const ERROR = { message: 'Use the new-style context.report() API.', type: 'Identifier' };
Copy link
Contributor Author

Choose a reason for hiding this comment

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

this was caught by the new eslint v7. 🎉

@aladdin-add aladdin-add changed the title wip!!!Upgrade: eslint v7 Upgrade: eslint v7 Apr 7, 2020
.eslintrc.yml Outdated
- error
- arrays: always-multiline
objects: always-multiline
functions: never # disallow trailing commases in function(es2017)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

you want to change your eslint-config? :)

@aladdin-add aladdin-add changed the title Upgrade: eslint v7 Upgrade: eslint and other deps Apr 7, 2020
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants