Skip to content

Implement minKeys option and update babel-eslint to @babel/eslint-parser #27

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

giacomocerquone
Copy link

@giacomocerquone giacomocerquone commented Apr 28, 2021

The install process was stuck due to a 404 while trying to fetch this package: eslint-plugin-tyrecheck.
I've removed babel-eslint since it's known to be deprecated and I've integrated @babel/eslint-parser instead which require a babel.config.js file instead.

I reintroduced the commented tests for the minKeys option

Test are passing

@giacomocerquone giacomocerquone changed the title Fix/add minkeys update babel eslint Implement minKeys option and update babel-eslint to @babel/eslint-parser Apr 28, 2021
@gszliu
Copy link

gszliu commented Aug 11, 2021

@leo-buneev Any followup for this? Seems like a useful change that could benefit a lot of repos.

@gszliu
Copy link

gszliu commented Aug 24, 2021

@leo-buneev Apologies for pinging again, would be great to get an update.

@gszliu
Copy link

gszliu commented Oct 8, 2021

@leo-buneev is anyone else available to maintain this package? It's incredibly useful and it would really be awesome if we could get a bit more oversight for pull requests

@CvBlixen
Copy link

CvBlixen commented May 12, 2022

I would love to see minKeys supported. 🙂

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