Skip to content

Conversation

@mmason2-godaddy
Copy link
Contributor

No description provided.

extends: ['eslint:recommended'],
parserOptions: {
ecmaVersion: 9,
ecmaVersion: 13,
Copy link
Contributor

Choose a reason for hiding this comment

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

Wow, this was pretty far behind.

Copy link
Contributor

Choose a reason for hiding this comment

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

Actually, before we merge... is ecmaVersion 13 available on eslint v7? We may need to bump the peerDependencies and as a result require a new major release of this config

Copy link
Contributor

Choose a reason for hiding this comment

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

@agerard-godaddy @mmason2-godaddy just as a follow up I am going to bump the peerDependency to v8 prior to release of a new major version

@mmason2-godaddy
Copy link
Contributor Author

I can't merge just FYI

@rmarkins-godaddy rmarkins-godaddy merged commit 8a31b31 into godaddy:main Dec 12, 2022
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