-
Notifications
You must be signed in to change notification settings - Fork 27.2k
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
switching channel prompts to run flutter upgrade #49511
switching channel prompts to run flutter upgrade #49511
Conversation
It looks like this pull request may not have tests. Please make sure to add tests before merging. If you need an exemption to this rule, contact Hixie. Reviewers: Read the Tree Hygiene page and make sure this patch meets those guidelines before LGTMing. |
Can this also close #14979 ? |
@jonahwilliams is this consistent with your plans for |
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.
I would adjust the phrasing slightly. Maybe "To ensure you're on the lastest build for this channel, run flutter upgrade"
There are some existing tests in https://github.com/flutter/flutter/blob/master/packages/flutter_tools/test/commands.shard/permeable/upgrade_test.dart that check if stdout contains the right strings, this addition should have a basic smoke test there |
@jonahwilliams but the changes are related to running |
Oh, sorry I wasn't thinking. It would go in the channel_test.dart file |
@jonahwilliams added a test |
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.
LGTM with nit
@jonahwilliams I added the complete line |
Thanks for the contribution! |
This pull request is not suitable for automatic merging in its current state.
|
Looks like this change broke another test:
@mannprerak2 can you take a look? |
Fixed.. I didn't know calls to verify were neccesary verify(mockProcessManager.start(
<String>['git', 'fetch'],
workingDirectory: anyNamed('workingDirectory'),
environment: anyNamed('environment'),
)).called(1); |
@jonahwilliams can you look into why tool_tests-commands-macos test is failing? |
a6fd06b
to
c5c047c
Compare
Description
Added print statements to prompt user to run
flutter upgrade
after runningflutter channel foo
Related Issues
resolves #16030
Tests
None... simply added print statements
Here's an output from my system:
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes (
[x]
). This will ensure a smooth and quick review process.///
).flutter analyze --flutter-repo
) does not report any problems on my PR.Breaking Change
Did any tests fail when you ran them? Please read Handling breaking changes.