Skip to content
This repository was archived by the owner on Dec 16, 2024. It is now read-only.

Comments

Remove steps.yml from saltfs#513

Closed
aneeshusa wants to merge 1 commit intoservo:masterfrom
aneeshusa:remove-steps-yml
Closed

Remove steps.yml from saltfs#513
aneeshusa wants to merge 1 commit intoservo:masterfrom
aneeshusa:remove-steps-yml

Conversation

@aneeshusa
Copy link
Contributor

@aneeshusa aneeshusa commented Oct 19, 2016

These now live in the main servo/servo repo.

r? @larsbergstrom
I'd like to have at least one PR build + be merged with the new configuration before removing the steps in case we need to revert/quickfix.


This change is Reviewable

These now live in the main servo/servo repo.
@larsbergstrom
Copy link
Contributor

@bors-servo r+

@bors-servo
Copy link
Contributor

📌 Commit 21da33f has been approved by larsbergstrom

@bors-servo
Copy link
Contributor

⌛ Testing commit 21da33f with merge 28b5b42...

bors-servo pushed a commit that referenced this pull request Oct 19, 2016
Remove steps.yml from saltfs

These now live in the main servo/servo repo.

r? @larsbergstrom
I'd like to have at least one PR build + be merged with the new configuration before removing the steps in case we need to revert/quickfix.

<!-- 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/513)
<!-- Reviewable:end -->
@larsbergstrom
Copy link
Contributor

@bors-servo r-

@larsbergstrom
Copy link
Contributor

larsbergstrom commented Oct 19, 2016

(Just noticed that we don't have a PR pending - I assumed ONE must have gone through since I rolled out the changes, but was wrong...)

@larsbergstrom
Copy link
Contributor

Hrm, yeah, there's a failure, but just on windows-dev:
http://build.servo.org/builders/windows-dev/builds/1568

@bors-servo
Copy link
Contributor

☀️ Test successful - status-travis

@aneeshusa
Copy link
Contributor Author

Taken care of in #522.

@aneeshusa aneeshusa closed this Oct 29, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants