-
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
[flutter_tools] update pubspec .yaml message should not be displayed when creating a new plugin #62599
Conversation
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
@@ -36,6 +37,7 @@ final Generator _kNoColorTerminalPlatform = () => FakePlatform.fromPlatform(cons | |||
final Map<Type, Generator> noColorTerminalOverride = <Type, Generator>{ | |||
Platform: _kNoColorTerminalPlatform, | |||
}; | |||
final BufferLogger logger = BufferLogger.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.
Add this in main()
under `LoggingProcessManager loggingProcessManager;
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.
done
@@ -49,6 +49,7 @@ void main() { | |||
Directory projectDir; | |||
FlutterVersion mockFlutterVersion; | |||
LoggingProcessManager loggingProcessManager; | |||
BufferLogger logger = BufferLogger.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.
Sorry, I also meant because logger = BufferLogger.test()
is in setUp, match the pattern of LoggingProcessManager
and just declare it here.
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.
Yeah I did that and forgot the remove the other constructor call. Fixed in a new commit.
@@ -49,6 +49,7 @@ void main() { | |||
Directory projectDir; | |||
FlutterVersion mockFlutterVersion; | |||
LoggingProcessManager loggingProcessManager; | |||
BufferLogger logger = BufferLogger.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.
BufferLogger logger = BufferLogger.test(); | |
BufferLogger logger; |
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.
done
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, thanks!
…when creating a new plugin (flutter#62599) * update pubspec message should not be displayed when creating a new plugin * formatting * revikew * review * cleanup
…when creating a new plugin (flutter#62599) * update pubspec message should not be displayed when creating a new plugin * formatting * revikew * review * cleanup
Description
Fix a bug where the update
pubspec.yaml
message always display even when creating a new plugin.Related Issues
fixes #62594
Tests
flutter create -t plugin in an empty folder should not show pubspec.yaml updating suggestion
flutter create -t plugin in an existing plugin should show pubspec.yaml updating suggestion
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.