-
Notifications
You must be signed in to change notification settings - Fork 4
Rollup and vite upgrade #358
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
Conversation
…or versions and adding and example to /examples/plugin/vite
|
Shouldn't we add this an example to the example repo? |
examples/plugin/vite/package.json
Outdated
| "typescript": "^5.0.4", | ||
| "vite": "7.1.5" | ||
| } | ||
| } No newline at end of file |
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.
end of file character - can we add an enter?
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.
Done
|
@konraddysput we didn't have any examples for the plugins, only sdks, so I created /examples/plugin. I can move it under /examples/sdk/ if you'd prefer though |
It does the same what our example do so maybe its just better to use an sdk example? Its up to you guys- its just an suggestion. |
|
I feel you. I do like that it's an example that's vite specific though. I'll probably keep it if you don't mind. |
Updating the rollup and vite plugins to be compatible with the latest major versions as well as added an example in /examples/plugin/vite