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

enable lint overridden_fields #9527

Merged
merged 2 commits into from
Apr 22, 2017
Merged

Conversation

a14n
Copy link
Contributor

@a14n a14n commented Apr 21, 2017

No description provided.

@abarth
Copy link
Contributor

abarth commented Apr 21, 2017

LGTM

@@ -105,7 +105,7 @@ class FlutterErrorDetails {

/// Converts the [exception] to a string.
///
/// This applies some additional logic to
/// This applies some additional logic to
Copy link
Contributor

Choose a reason for hiding this comment

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

Please fix my work for me and complete this sentence :-)

This applies some additional logic to make [AssertionError] exceptions prettier, to handle exceptions that stringify to empty strings, to handle objects that don't inherit from [Exception] or [Error], and so forth.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@a14n a14n merged commit ad496e1 into flutter:master Apr 22, 2017
@a14n a14n deleted the enable-lint-overridden_fields branch April 22, 2017 08:27
@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.

None yet

4 participants