-
-
Notifications
You must be signed in to change notification settings - Fork 26
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 support for sonarjs #132
Add support for sonarjs #132
Conversation
684f656
to
0ee8041
Compare
@Julien-R44 There is also another related eslint plugin https://github.com/un-ts/eslint-plugin-sonar |
Thanks for sharing, I didn't know. Would also be cool to have |
@@ -55,6 +55,7 @@ | |||
"@types/node": "~18.7.23", | |||
"@types/prettier": "~2.7.1", | |||
"@typescript-eslint/eslint-plugin": "~5.38.1", | |||
"@typescript-eslint/experimental-utils": "^5.38.1", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is this needed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can't remember why ahah. I'm sorry for the delay @Shinigami92 I have a lot of stuff to do right now 🥲
I think I'll get back to the project in a week or two and resume theses PR
However, what I wanted to do was to finish our patch module, and once merged, regenerate the types for the plugins I proposed PRs for
https://github.com/SonarSource/eslint-plugin-sonarjs
Related to #131