Skip to content

/ servo

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

Jump to bottom

Failures of tidy aren't actually causing builds to fail #3482

Closed
zwarich opened this issue Sep 25, 2014 · 1 comment
Closed

Failures of tidy aren't actually causing builds to fail #3482

zwarich opened this issue Sep 25, 2014 · 1 comment
Labels
A-infrastructure

Comments

@zwarich
Copy link

@zwarich zwarich commented Sep 25, 2014

Since switching to Cargo / Mach, tidy failures aren't actually causing builds to fail.
@zwarich zwarich mentioned this issue Oct 6, 2014
Merged
bors-servo pushed a commit that referenced this issue Oct 6, 2014
bors-servo
auto merge of #3592 : zwarich/servo/trailing-whitespace, r=Manishearth
Loading status checks…
3dacbfa 
The Mach test runner doesn't actually make these failures yet, which is
tracked by #3482.
bors-servo pushed a commit that referenced this issue Oct 7, 2014
bors-servo
auto merge of #3592 : zwarich/servo/trailing-whitespace, r=Manishearth
Loading status checks…
968cc18 
The Mach test runner doesn't actually make these failures yet, which is
tracked by #3482.
bors-servo pushed a commit that referenced this issue Oct 7, 2014
bors-servo
auto merge of #3592 : zwarich/servo/trailing-whitespace, r=Manishearth
Loading status checks…
6bbd3f4 
The Mach test runner doesn't actually make these failures yet, which is
tracked by #3482.
bors-servo pushed a commit that referenced this issue Oct 7, 2014
bors-servo
auto merge of #3592 : zwarich/servo/trailing-whitespace, r=Manishearth
Loading status checks…
ae946a9 
The Mach test runner doesn't actually make these failures yet, which is
tracked by #3482.
@larsbergstrom
Copy link
Contributor

@larsbergstrom larsbergstrom commented Jan 8, 2015

I believe that these are now causing failures (we've seen them break commits now).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-infrastructure
Projects
None yet
Milestone
No milestone
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
@jdm @zwarich @larsbergstrom
You can’t perform that action at this time.