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

feat(eslint): add eslint-plugin-qwik #90

Closed
wants to merge 3 commits into from

Conversation

otodockal
Copy link
Contributor

@otodockal otodockal commented Nov 23, 2021

MVP

  • unit tests integration with Jest
  • eslint 8.x
  • TS
  • publish eslint-plugin-qwik to NPM

the implementation follows these spec rules https://gist.github.com/mhevery/f56cd107bd358625a0a2753eba63147a#file-spec-tsx

@mhevery
Copy link
Contributor

mhevery commented Jul 12, 2022

@otodockal I am going to close this as we now have it implemented here: https://github.com/BuilderIO/qwik/tree/main/packages/eslint-plugin-qwik

I know that you worked very hard on this, and I want to thank you for your contribution. Sorry that your work did not make it in as we have decided to go a slightly different route. But thank you.

@mhevery mhevery closed this Jul 12, 2022
@otodockal
Copy link
Contributor Author

Yeah, I see @mhevery, it's OK, I could learn something new. Either way, the new API feels way more ergonomic. This rule is very strict, so it's better not to need it, definitely.

@shairez
Copy link
Contributor

shairez commented Jul 13, 2022

Thanks for the feedback @otodockal ! (and for your contribution) 🙏

@cloudflare-pages
Copy link

cloudflare-pages bot commented Sep 28, 2022

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: cee2ad3
Status: ✅  Deploy successful!
Preview URL: https://f7bce8ff.qwik-docs.pages.dev

View logs

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.

None yet

4 participants