Skip to content
This repository has been archived by the owner on Nov 1, 2024. It is now read-only.

Run dartfmt to fix the build. #102

Merged
merged 2 commits into from
Mar 31, 2017
Merged

Run dartfmt to fix the build. #102

merged 2 commits into from
Mar 31, 2017

Conversation

matanlurey
Copy link
Contributor

No description provided.

natebosch
natebosch approved these changes Mar 31, 2017
View reviewed changes
Copy link
Contributor

@natebosch natebosch left a comment

Choose a reason for hiding this comment

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

Curious - Was this a change due to upgrading dartfmt, or was this file never fully consistent against any version of dartfmt?

@matanlurey
Copy link
Contributor Author

@natebosch I think dartfmt changed :-/

@matanlurey matanlurey merged commit 429e5d1 into dart-archive:master Mar 31, 2017
@matanlurey matanlurey deleted the fix-travis branch March 31, 2017 00:16
@zoechi
Copy link
Contributor

zoechi commented Mar 31, 2017

The change looks more like a dartfmt regression to me than an improvment

mosuem pushed a commit to dart-lang/tools that referenced this pull request Oct 25, 2024
@matanlurey
Run dartfmt to fix the build. (dart-archive/code_builder#102)
f98d82a 
* Dartfmt.

* Remove dev.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@natebosch natebosch natebosch approved these changes

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@matanlurey @zoechi @natebosch @googlebot