-
-
Notifications
You must be signed in to change notification settings - Fork 20
feat: refresh project dependencies #66
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
this also changes `module` to esnext. We are using esbuild to build, so between modern flags `nodenext` and `bundler` we should pick `bundler`
This also has a small refactoring of the package name prompt to handle modern requirements
Latest node allows to use --run flag to launch scripts faster
…youts-next The main goal is to get latest version of deps installed. This also comes with migrating to vite-plugin-vue-layouts-next from discontinued vite-plugin-vue-layouts. Eslint integration was also updated
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.
Looking really good. Only a few points of feedback.
Converting it to draft, I would like to merge #67 and use our refreshed eslint config first |
56df0dd
to
d549446
Compare
Update the vuetify dependency in both TypeScript and JavaScript template package.json files to version 3.8.0 to ensure compatibility and leverage the latest features and improvements
Eslint config adopted, only wait for #67 to be finalised |
argg, I must be blind, I didn't see this PR |
I'll prioritize this. |
ad7d447
to
7537054
Compare
Update sass-embedded dependency to ^1.86.3 across multiple templates, drop sass dependency and add preprocessorOptions for scss
This also updates sorting order and default package manager
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.
Tentative approval. I have to run through all the variations still, but from a code perspective, everything seems to be in order.
To address modern pnpm policy (https://pnpm.io/cli/approve-builds) from poc was backported util to check ignored builds
Enable automatic imports for Pinia functions (defineStore, useStore, storeToRefs) in both JavaScript and TypeScript templates to simplify usage and reduce manual imports
Simplify auto-imports configuration by using recommended VueRouterAutoImports and removing non-existing Pinia imports.
In this PR we have:
vite-plugin-vue-layouts
tovite-plugin-vue-layouts-next
(see comment)closes #65
related #54