Skip to content

Update analysis options - #8203

Merged
tvolkert merged 1 commit into
flutter:masterfrom
tvolkert:analyzer
Feb 15, 2017
Merged

tvolkert merged 1 commit into
flutter:masterfrom
tvolkert:analyzer

Conversation

@tvolkert

Copy link
Copy Markdown
Contributor

Fixes #7166

Also cleans up from an aborted attempt at #5793

@tvolkert

Copy link
Copy Markdown
Contributor Author

@danrubel

@tvolkert
tvolkert merged commit 7b8705f into flutter:master Feb 15, 2017
@tvolkert
tvolkert deleted the analyzer branch February 15, 2017 22:41
@Hixie

Hixie commented Mar 20, 2017

Copy link
Copy Markdown
Contributor

This change was only applied to one of the three .analysis_options files. @tvolkert, can you update the others to make sure they're all in sync (or that if they're different, it's intentional)?

@tvolkert

Copy link
Copy Markdown
Contributor Author

This change was only applied to one of the three .analysis_options files

All the other .analysis_options files (e.g. this one) are just skeletons...

@Hixie

Hixie commented Mar 28, 2017

Copy link
Copy Markdown
Contributor

I meant the three that are mentioned at the top of the root one:
https://github.com/flutter/flutter/blob/master/.analysis_options

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Aug 13, 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.

Add a "blacklisted methods" test

4 participants