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

@vue/cli: init at 3.0.1 #45456

Closed
wants to merge 1 commit into from
Closed

@vue/cli: init at 3.0.1 #45456

wants to merge 1 commit into from

Conversation

tiramiseb
Copy link
Contributor

Motivation for this change

Vue CLI v3 stable has been released.

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Fits CONTRIBUTING.md.

Some other Node packages have been upgraded when executing generate.sh. I guess it is okay to include these changes in the PR (because well, It's hard to make sure what is needed by Vue CLI and what has just been upgraded because a new version exists), but I'm not 100% sure.

@@ -14,6 +14,7 @@
, "sloc"
, "vue-cli"
, "swagger"
, "@vue/cli"
Copy link
Member

@Mic92 Mic92 Aug 22, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cc @svanderburg what is the preferred package set for node packages? We have apparently v6, v8, v10. I would prefer if most packages would be included only once in one of the node versions, due the size of these files.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If v6 is end-of-life it should be also removed all together.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah I agree on that. Maybe move everything listed in node-packages-v6.json to node-packages-v10.json so we don't lose anything. My understanding is that node versions are close enough to not be a huge issue for most use cases.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here it says we should stick to the latest LTS: https://github.com/NixOS/nixpkgs/blob/master/doc/languages-frameworks/node.section.md which is 8_x

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

April 2019 means that we need to upgrade before the next release.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

v8 upgrade... #45700

@@ -14,6 +14,7 @@
, "sloc"
, "vue-cli"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So vue-cli is the old @vue/cli?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@vue/cli is Vue CLI v3
vue-cli is Vue CLI v2

Even though v2 is deprecated, most users still use it, we must give them enough time to switch...

Copy link
Member

@Mic92 Mic92 Aug 28, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok. Is it possible to add it to node-packages-v6? How can the resulting package be used?

Copy link
Contributor Author

@tiramiseb tiramiseb Aug 28, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

From the vue-cli v3 documentation: "Vue CLI requires Node.js version 8.9 or above"

@Mic92
Copy link
Member

Mic92 commented Aug 28, 2018

This pull request will be part of #45700

@Mic92 Mic92 closed this in ca0260e Sep 1, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants