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

Add supports for eslint-plugin-unicorn #55

Closed
fisker opened this issue Jul 28, 2021 · 12 comments
Closed

Add supports for eslint-plugin-unicorn #55

fisker opened this issue Jul 28, 2021 · 12 comments
Labels
enhancement New feature or request

Comments

@fisker
Copy link
Collaborator

fisker commented Jul 28, 2021

Wow, found this is so cool, can we add support for eslint-plugin-unicorn?

@Shinigami92 Shinigami92 added the enhancement New feature or request label Jul 28, 2021
@Shinigami92
Copy link
Collaborator

can we add support for eslint-plugin-unicorn

Sure! It is highly extendable 🙂

There is already support for e.g. jsdoc and other stuff.

Do you want to try it and open a PR?

Will grant you write access so you don't need to fork

There is also a script to generate a rule

https://github.com/Shinigami92/eslint-define-config/blob/435ca836125e7228ad8afce9ee3909207c309eb1/package.json#L13

@fisker
Copy link
Collaborator Author

fisker commented Jul 28, 2021

Do you want to try it and open a PR?

Sorry, but I don't use ts at all.

@Shinigami92
Copy link
Collaborator

Sorry, but I don't use ts at all.

I will not hurt you 😏
Or will it? 👀


Could you write your top 10 most used rules of eslint-plugin-unicorn?
Then I could add these.
The generation is not that automated 🙁 So I need to hand-write every rule on its own...
That's why I mostly support first the rules that are common used.

@fisker
Copy link
Collaborator Author

fisker commented Jul 28, 2021

I had a quick look on the script, you're hard coding the docs links, why not load the rule, links exists in their meta.

Can these options generated from the schema?(Just an idea.)

@Shinigami92
Copy link
Collaborator

I had a quick look on the script, you're hard coding the docs links, why not load the rule, links exists in their meta.

Can these options generated from the schema?(Just an idea.)

Every plugin has a different kind of structure and generating them automatically would not result in such good types than with handcrafted types and JSDoc.

@fisker
Copy link
Collaborator Author

fisker commented Jul 28, 2021

Every plugin has a different kind of structure

Can you explain?

@fisker
Copy link
Collaborator Author

fisker commented Jul 28, 2021

I see, but we can still use script to generate them, and modify if nessary.

@Shinigami92
Copy link
Collaborator

Please feel free to create a draft. And feel free to do that in plain js for now.
If it works I can convert it to ts later on.

Just create a branch/PR and feel free to escalate 😄

@fisker
Copy link
Collaborator Author

fisker commented Jul 28, 2021

Okay, I'll try, going to sleep now.

@Shinigami92
Copy link
Collaborator

Okay, I'll try, going to sleep now.

Added you on Discord 👀

@Shinigami92
Copy link
Collaborator

Finally I found some free time to work on an automation to generate the rules

I added support for your requested plugin 🎉

cabba8d

Will be released in 1.1.0

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

No branches or pull requests

2 participants