Skip to content

Update upgrading.md#270

Merged
sethladd merged 3 commits into
masterfrom
danrubel-patch-1
Aug 31, 2016
Merged

Update upgrading.md#270
sethladd merged 3 commits into
masterfrom
danrubel-patch-1

Conversation

@danrubel

Copy link
Copy Markdown

Addresses doc request in flutter/flutter#4994
@sethladd

Comment thread upgrading.md Outdated

If you've modified your `pubspec.yaml` file, or you want to update
the packages you app depends upon without updating flutter itself,
use...

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Normally, I believe we use : instead of ...

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Done.

@sethladd

Copy link
Copy Markdown
Contributor

Thanks!

cc @qchong for the review

Comment thread upgrading.md
@@ -17,7 +17,15 @@ command from the root of your app (the same directory that contains the
$ flutter upgrade

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Does this also run flutter packages upgrade ?

Or is it now purely scoped to just upgrading the flutter SDK associated with this app?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

flutter upgrade updates both channel and packages as before, per your recommendation.

@sethladd Comments addressed. PTAL.
Comment thread upgrading.md Outdated

To update, use the `flutter upgrade`
## Upgrading flutter channel and your packages
To update both the flutter sdk and your packages, use the `flutter upgrade`

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

s/flutter sdk/Flutter SDK

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Done.

@sethladd

Copy link
Copy Markdown
Contributor

lgtm with capitalization fixes

thanks!

@danrubel

Copy link
Copy Markdown
Author

Thank you both for the comments. Fixed.

@sethladd

Copy link
Copy Markdown
Contributor

Thanks!

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.

3 participants