-
Notifications
You must be signed in to change notification settings - Fork 2
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
SWC plugin for Vite and NextJS #186
SWC plugin for Vite and NextJS #186
Conversation
@@ -0,0 +1,11 @@ | |||
# react-obsidian |
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.
Do you plan on adding short docs or description here? If not I think we can remove the file. Even 1-2 lines is enough, we'll document this in the installation section.
{ | ||
"name": "swc-plugin-obsidian", | ||
"version": "3.0.0-alpha.6", | ||
"description": "Dependency injection framework for React and React Native applications", |
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.
Don't forget to update the description.
"build": "tsc --project tsconfig.json", | ||
"pretest": "tsc --project tsconfig.json", | ||
"test": "npx jest", | ||
"test-coverage": "npx jest --coverage" |
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.
We don't have unit tests so I guess we can remove the coverage script?
"globals": "^15.9.0", | ||
"jest": "29.7.x", | ||
"jest-environment-jsdom": "^29.7.0", | ||
"jest-extended": "^4.0.0", |
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.
Some of these dependencies aren't used - we should clean up the dev dependencies later. I'm pretty sure jest-extended
and jest-when
aren't used.
Closing in favor of #190 . Thanks @zacharyweidenbach for your hard work 👑 |
implementation, fails to compile due to module compat issue