Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Fix: update ecmaVersion 8 to 2018 in recommended configuration #122

Closed
wants to merge 1 commit into from
Closed

Fix: update ecmaVersion 8 to 2018 in recommended configuration #122

wants to merge 1 commit into from

Conversation

ybiquitous
Copy link
Contributor

Hi!

This PR fixes {ecmaVersion: 8} to {ecmaVersion: 2018}, which is different from the latest README.

Thanks.

@codecov-io
Copy link

codecov-io commented Jun 17, 2018

Codecov Report

Merging #122 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #122   +/-   ##
=======================================
  Coverage   95.97%   95.97%           
=======================================
  Files          43       43           
  Lines        1117     1117           
=======================================
  Hits         1072     1072           
  Misses         45       45

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e957849...a6ba2b2. Read the comment docs.

@mysticatea
Copy link
Owner

I apologize for the delay.
I have updated the recommended config with ecmaVersion: 2019.

Thank you for contributing!

@mysticatea mysticatea closed this Jul 16, 2018
@ybiquitous
Copy link
Contributor Author

OK, I'll try v7!

brettz9 pushed a commit to brettz9/eslint-plugin-node that referenced this pull request Jul 24, 2024
…list (mysticatea#123) (fixes mysticatea#122)

Co-authored-by: 唯然 <weiran.zsd@outlook.com>
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.

3 participants