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

Backet misspelling? #67

Open
wSedlacek opened this issue Aug 26, 2021 · 3 comments
Open

Backet misspelling? #67

wSedlacek opened this issue Aug 26, 2021 · 3 comments
Labels

Comments

@wSedlacek
Copy link

wSedlacek commented Aug 26, 2021

CommaOrCloseBacketExpected: 0x205,

I was just doing a spelling audit of my code and I noticed in my eslint config that I had json/comma-or-close-backet-expected.
Should this be bracket instead of backet?
It would be a breaking change to change the name of the rule, but thought I make make an issue anyways to see if perhaps it should be corrected in the next major release.

@azeemba
Copy link
Owner

azeemba commented Aug 27, 2021

Whoops that's embarrassing. Thanks for reporting this. We will see if we can address it in the next major release.

@AdrieanKhisbe
Copy link
Collaborator

oopsie 🙈

@azeemba maybe we can already introduce the json/comma-or-close-bracket-expected in a minor.
What do you think?

@azeemba
Copy link
Owner

azeemba commented Aug 27, 2021

With a quick glance of the code, I wasn't sure if we could map the same error code to two different names without breaking existing configurations.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants