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

updated package.json and package-lock.json to still allow this template to work even when node >=18 is installed #4

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

11BelowStudio
Copy link

@11BelowStudio 11BelowStudio commented Oct 20, 2023

Incompatible with #5 and #6 (#5 and #6 are much less hacky though)

Added node 16.20.2 as a dependency. This is because @vue/cli-shared-utils 4.5.19 (dependency of the various @vue/cli dependencies) requires @achrinza/node-ipc 9.2.2, which won't work with anything newer than 17 - rendering this template unusable if you have node 18 (or later) installed on your system (unless you faff around with a node version manager).

Opted to depend on the latest version of node 16 instead of 17 because 16 was an LTS - therefore probably safer to use I guess (although it's still EOL).

Also updated the lockfile as well, to make it even easier to get up and running for the end user.

This is an incredibly hacky workaround - but it looks like the only other option would be to do a lot of faffing around with all the dependencies allowing the vue cli utils to be updated (in turn using a node-ipc version that works on node >=18) - or just migrate from vue cli to Vite - or maybe go all-in and migrate the project from Vue 2 to Vue 3. However, I have 0 prior experience with Vue (or understanding of how any of this works), so chances are that any attempts I may make at migrating it would end very badly. So, in the meantime, I'll just contribute this workaround.

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.

1 participant