Skip to content
This repository has been archived by the owner on Feb 22, 2023. It is now read-only.

remove unnecessary new from samples #818

Merged
merged 1 commit into from
Oct 16, 2018

Conversation

a14n
Copy link
Contributor

@a14n a14n commented Oct 5, 2018

No description provided.

@a14n a14n requested a review from mit-mit October 5, 2018 16:03
@mit-mit
Copy link
Member

mit-mit commented Oct 8, 2018

Analysis issues around constructor order are also showing up in head, and look related to a Dart roll, not this PR.

@mit-mit
Copy link
Member

mit-mit commented Oct 8, 2018

LGTM, but let's get the CI job working in head before we merge this

@mit-mit
Copy link
Member

mit-mit commented Oct 8, 2018

cc @cbracken is someone looking at the CI issue?

@a14n
Copy link
Contributor Author

a14n commented Oct 8, 2018

I'm fixing the CI issues.

@a14n a14n force-pushed the unnecessary_new_sample branch from 84e34e5 to ffbf999 Compare October 8, 2018 19:57
@a14n
Copy link
Contributor Author

a14n commented Oct 9, 2018

As the build failures are now the same as in master (and as this PR only touch READMEs and docs) I'm tempted to merge this PR.

WDYT?

@mit-mit
Copy link
Member

mit-mit commented Oct 9, 2018

@cbracken can we land this?

@mit-mit
Copy link
Member

mit-mit commented Oct 16, 2018

Merging as CI is passing now

Copy link
Member

@mit-mit mit-mit left a comment

Choose a reason for hiding this comment

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

LGTM

@mit-mit mit-mit merged commit 60eeff0 into flutter:master Oct 16, 2018
@a14n a14n deleted the unnecessary_new_sample branch October 16, 2018 13:12
@a14n
Copy link
Contributor Author

a14n commented Oct 16, 2018

Thanks

@mit-mit
Copy link
Member

mit-mit commented Oct 16, 2018

@a14n looks like the CI job failed after the merge; does this look related to your changes?
https://cirrus-ci.com/build/6241182061428736

@a14n
Copy link
Contributor Author

a14n commented Oct 16, 2018

I don't think it's related to my PR (only README and comments had been changed).

andreidiaconu pushed a commit to andreidiaconu/plugins that referenced this pull request Feb 17, 2019
andreidiaconu added a commit to andreidiaconu/plugins that referenced this pull request Feb 17, 2019
julianscheel pushed a commit to jusst-engineering/plugins that referenced this pull request Mar 11, 2020
Akachu pushed a commit to Akachu/flutter_camera that referenced this pull request Apr 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants