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

Prepare for v2.5.0 release. #3235

Merged
merged 3 commits into from
Aug 3, 2021
Merged

Conversation

terrylucas
Copy link
Contributor

No description provided.

@terrylucas terrylucas requested review from devoncarew and kenzieschmoll and removed request for devoncarew August 2, 2021 21:21
@@ -2,7 +2,7 @@ name: devtools_repo
description: A repo management tool for DevTools.

environment:
sdk: '>=2.12.0 <3.0.0'
Copy link
Member

@kenzieschmoll kenzieschmoll Aug 2, 2021

Choose a reason for hiding this comment

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

is this change necessary? looks like this may be what is causing the bots to fail

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Certainly caused bots to not work. However, update_version.sh would not run without the sdk change e.g.


tool/update_version.dart:50:7: Error: Null safety features are disabled for this library.
Try removing the package language version or setting the language version to 2.12 or higher.
String? incrementVersion(String oldVersion) {

@terrylucas terrylucas merged commit 89e2169 into flutter:master Aug 3, 2021
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.

2 participants