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

Add stylo unit tests to linux-dev #481

Merged
merged 1 commit into from Sep 30, 2016
Merged

Conversation

@Manishearth
Copy link
Member

Manishearth commented Sep 21, 2016

r? @larsbergstrom


This change is Reviewable

@larsbergstrom
Copy link
Contributor

larsbergstrom commented Sep 21, 2016

@bors-servo
Copy link
Contributor

bors-servo commented Sep 21, 2016

📌 Commit 7c155d7 has been approved by larsbergstrom

@bors-servo
Copy link
Contributor

bors-servo commented Sep 21, 2016

Testing commit 7c155d7 with merge f405340...

bors-servo added a commit that referenced this pull request Sep 21, 2016
Add stylo unit tests to linux-dev

r? @larsbergstrom

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/saltfs/481)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Sep 21, 2016

💔 Test failed - status-travis

@Manishearth
Copy link
Member Author

Manishearth commented Sep 21, 2016

@bors-servo retry

CommandExecutionError: Command 'git config homebrew.analyticsdisabled "true"' failed. Stderr: 'error: could not lock config file .git/config: No such file or directory'

Not sure what this error is; might be a travis intermittent?

bors-servo added a commit that referenced this pull request Sep 21, 2016
Add stylo unit tests to linux-dev

r? @larsbergstrom

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/saltfs/481)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Sep 21, 2016

Testing commit 7c155d7 with merge 2985817...

@bors-servo
Copy link
Contributor

bors-servo commented Sep 21, 2016

💔 Test failed - status-travis

@aneeshusa
Copy link
Member

aneeshusa commented Sep 21, 2016

Just a hunch, but likely related to Homebrew 1.0 and their migration of the homebrew git dir. Will investigate after class. (Also see comments about robustness at #361).

@Manishearth
Copy link
Member Author

Manishearth commented Sep 21, 2016

thanks!

@aneeshusa
Copy link
Member

aneeshusa commented Sep 22, 2016

Watch #483 for a solution to the Homebrew error.

@aneeshusa
Copy link
Member

aneeshusa commented Sep 30, 2016

@bors-servo retry

  • #483, #492 are merged and OS X builders are green on Travis again
@bors-servo
Copy link
Contributor

bors-servo commented Sep 30, 2016

Testing commit 7c155d7 with merge a4b6090...

bors-servo added a commit that referenced this pull request Sep 30, 2016
Add stylo unit tests to linux-dev

r? @larsbergstrom

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/saltfs/481)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Sep 30, 2016

☀️ Test successful - status-travis

@bors-servo bors-servo merged commit 7c155d7 into servo:master Sep 30, 2016
1 of 2 checks passed
1 of 2 checks passed
continuous-integration/travis-ci/pr The Travis CI build failed
Details
homu Test successful
Details
@Manishearth Manishearth deleted the Manishearth:stylo-unit branch Oct 1, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.