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.

Sign up for GitHub

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

Jump to bottom

Run 'pub get' the first time the tests are run #66

Merged
merged 1 commit into from
Nov 10, 2015
Merged

Run 'pub get' the first time the tests are run #66

merged 1 commit into from
Nov 10, 2015
43 −14

Conversation

Hixie
Copy link
Contributor

@Hixie Hixie commented Nov 9, 2015

No description provided.

abarth
abarth reviewed Nov 9, 2015
View reviewed changes
bin/cache/README.md
@@ -1,3 0,0 @@
# Binary cache
Copy link
Contributor

Choose a reason for hiding this comment

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

Did you delete this README file on purpose?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

abarth
abarth reviewed Nov 9, 2015
View reviewed changes
packages/flutter_tools/lib/src/commands/test.dart
return 1;
}

if (!FileSystemEntity.isFileSync(path.join(flutterDir.path, 'pubspec.lock'))) {
Copy link
Contributor

Choose a reason for hiding this comment

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

You want to compare the mtime between these two files...

@abarth
Copy link
Contributor

abarth commented Nov 9, 2015

LGTM

@Hixie
Copy link
Contributor Author

Hixie commented Nov 9, 2015

@abarth ready for another look.

@abarth
Copy link
Contributor

abarth commented Nov 9, 2015

LGTM

Hixie added a commit that referenced this pull request Nov 10, 2015
@Hixie
Merge pull request #66 from Hixie/better-tests
0a1385d 
Run 'pub get' the first time the tests are run
@Hixie Hixie merged commit 0a1385d into flutter:master Nov 10, 2015
@Hixie Hixie deleted the better-tests branch November 10, 2015 20:26
@mpcomplete mpcomplete mentioned this pull request Apr 20, 2016
Closed
@mpcomplete mpcomplete mentioned this pull request Jun 17, 2016
Closed
@jason-simmons jason-simmons mentioned this pull request Jul 21, 2016
Closed
@mswift42 mswift42 mentioned this pull request Nov 13, 2016
Closed
@gavindoughtie gavindoughtie mentioned this pull request Dec 2, 2016
Closed
@ppolasek ppolasek mentioned this pull request Feb 8, 2017
Closed
@skybrian skybrian mentioned this pull request Feb 16, 2017
Closed
@Reagankm Reagankm mentioned this pull request Feb 24, 2017
Closed
@collinjackson collinjackson mentioned this pull request May 19, 2017
Closed
@jamesgopinath jamesgopinath mentioned this pull request Aug 4, 2017
Closed
@apwilson apwilson mentioned this pull request Aug 10, 2017
Closed
@JFixby JFixby mentioned this pull request Sep 7, 2017
Closed
@gavindoughtie gavindoughtie mentioned this pull request Sep 25, 2017
Closed
@arnold-nonstop arnold-nonstop mentioned this pull request Sep 29, 2017
Closed
@amirh amirh mentioned this pull request Oct 9, 2017
Closed
@PlanovskyJus PlanovskyJus mentioned this pull request Nov 3, 2017
Closed
@steele232 steele232 mentioned this pull request Nov 9, 2017
Closed
@ummuhan ummuhan mentioned this pull request Oct 6, 2020
Closed
@JeetBhattachariya JeetBhattachariya mentioned this pull request Oct 8, 2020
Closed
This was referenced Oct 16, 2020
Closed
Closed
Closed
@lv823063712 lv823063712 mentioned this pull request Oct 19, 2020
Closed
@right7ctrl right7ctrl mentioned this pull request Oct 24, 2020
Closed
@hasanworks hasanworks mentioned this pull request Oct 30, 2020
Closed
@Tony-Akani Tony-Akani mentioned this pull request Nov 1, 2020
Closed
@kw2019ltd kw2019ltd mentioned this pull request Nov 16, 2020
Closed
@page-bunnii page-bunnii mentioned this pull request Nov 20, 2020
Closed
@issantech issantech mentioned this pull request Nov 23, 2020
Closed
This was referenced Nov 28, 2020
Closed
Closed
This was referenced Dec 11, 2020
Closed
Closed
@sureace sureace mentioned this pull request Jan 13, 2021
Closed
@staminna staminna mentioned this pull request Jan 22, 2021
Closed
@yannainghtun-mk yannainghtun-mk mentioned this pull request Jan 26, 2021
Closed
@1005274291 1005274291 mentioned this pull request Jan 26, 2021
Closed
@praneethpj praneethpj mentioned this pull request Jan 29, 2021
Closed
@sravanpabolu sravanpabolu mentioned this pull request Feb 11, 2021
Closed
@lianchengtong lianchengtong mentioned this pull request Feb 13, 2021
Closed
@Marko1994 Marko1994 mentioned this pull request Mar 4, 2021
Closed
@bladeofgod bladeofgod mentioned this pull request Apr 14, 2021
Open
@SlowGen SlowGen mentioned this pull request May 27, 2021
Closed
@akashgk akashgk mentioned this pull request Jun 18, 2021
Closed
@SamStrike0809 SamStrike0809 mentioned this pull request Jun 18, 2021
Closed
@94wogus 94wogus mentioned this pull request Jul 3, 2021
Open
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 17, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Hixie @abarth