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

Added basic upgrading instructions #5176

Merged
merged 3 commits into from Feb 15, 2020
Merged

Added basic upgrading instructions #5176

merged 3 commits into from Feb 15, 2020

Conversation

Uninen
Copy link
Contributor

@Uninen Uninen commented Feb 10, 2020

What kind of change does this PR introduce? (check at least one)

  • Bugfix
  • Feature
  • Code style update
  • Refactor
  • Docs
  • Underlying tools
  • Other, please describe:

Does this PR introduce a breaking change? (check one)

  • Yes
  • No

Other information:

Fixes #5155

@Uninen
Copy link
Contributor Author

Uninen commented Feb 10, 2020

The first stab at adding upgrade instructions adds two parts 1) upgrading your global packages, and 2) upgrading packages inside your project. This distinction seems to be not very clear to all new users.

What I'd like to add also, but did not add yet, is an additional comment about upgrades that break (which again seems to be a somewhat usual but many times easily fixable situation) with something in the lines of:

"If there is an error with the upgrade, a clean install by manually removing and then installing may resolve the issue.

npm uninstall -g @vue/cli
npm install -g @vue/cli
# OR
yarn global remove @vue/cli
yarn global add @vue/cli

"

Copy link
Member

@NataliaTepluhina NataliaTepluhina left a comment

Choose a reason for hiding this comment

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

@Uninen thanks for this addition to the docs! I've added a few minor suggestions 😉

docs/guide/installation.md Outdated Show resolved Hide resolved
docs/guide/installation.md Outdated Show resolved Hide resolved
Uninen and others added 2 commits February 12, 2020 19:37
Co-Authored-By: Natalia Tepluhina <tarya.se@gmail.com>
Co-Authored-By: Natalia Tepluhina <tarya.se@gmail.com>
@NataliaTepluhina
Copy link
Member

Thanks for the changes!

@NataliaTepluhina NataliaTepluhina merged commit 677e4c5 into vuejs:master Feb 15, 2020
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

2 participants