Skip to content

Always run the flutter tool in checked mode. - #7655

Closed
Hixie wants to merge 1 commit into
flutter:masterfrom
Hixie:flutter-checked
Closed

Hixie wants to merge 1 commit into
flutter:masterfrom
Hixie:flutter-checked

Conversation

@Hixie

@Hixie Hixie commented Jan 26, 2017

Copy link
Copy Markdown
Contributor

No description provided.

@abarth

abarth commented Jan 26, 2017

Copy link
Copy Markdown
Contributor

Maybe we should do this on Travis? It seems wrong to run in checked mode in production.

@Hixie

Hixie commented Jan 26, 2017

Copy link
Copy Markdown
Contributor Author

I don't think we should keep this on forever, but we are regularly failing to catch problems because of not using checked mode around the team. Case in point, Travis failed this. Looking into it now...

@Hixie

Hixie commented Jan 26, 2017

Copy link
Copy Markdown
Contributor Author

Fixes #4807

@Hixie

Hixie commented Jan 26, 2017

Copy link
Copy Markdown
Contributor Author

Blocked on dart-lang/sdk#28519

@Hixie

Hixie commented Feb 1, 2017 •

Copy link
Copy Markdown
Contributor Author

Closing until dart-lang/sdk#28519 is fixed and rolled in.

@Hixie Hixie closed this Feb 1, 2017
@Hixie
Hixie deleted the flutter-checked branch February 5, 2017 21:05
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Aug 14, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants