Skip to content

/ saltfs

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

Remove unused cargo builders. #293

Merged
merged 1 commit into from Apr 4, 2016
Merged

Remove unused cargo builders. #293

merged 1 commit into from Apr 4, 2016

Conversation

@Ms2ger
Copy link
Contributor

Ms2ger commented Apr 4, 2016

All repositories that used those builders were moved to Travis.

This change is Reviewable
@Ms2ger
Remove unused cargo builders.
Loading status checks…
8a466c4 
All repositories that used those builders were moved to Travis.
@metajack
Copy link
Contributor

metajack commented Apr 4, 2016

@bors-servo r+

It's easy to bring this back if we ever need it again.

Review status: 0 of 1 files reviewed at latest revision, all discussions resolved.

Comments from Reviewable
@bors-servo
Copy link
Contributor

bors-servo commented Apr 4, 2016

📌 Commit 8a466c4 has been approved by metajack
@bors-servo
Copy link
Contributor

bors-servo commented Apr 4, 2016

⌛ Testing commit 8a466c4 with merge cad9cf9...
bors-servo added a commit that referenced this pull request Apr 4, 2016
@bors-servo
Auto merge of #293 - servo:cargo, r=metajack
Loading status checks…
cad9cf9 
Remove unused cargo builders.

All repositories that used those builders were moved to Travis.

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

bors-servo commented Apr 4, 2016

☀️ Test successful - travis
@bors-servo bors-servo merged commit 8a466c4 into master Apr 4, 2016
3 of 4 checks passed
3 of 4 checks passed
code-review/reviewable Review in progress: 0 of 1 files reviewed, all discussions resolved
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
homu Test successful
Details
@aneeshusa
Copy link
Member

aneeshusa commented Apr 4, 2016

:) I was wondering if we were still using these, I saw they hadn't had builds scheduled on build.servo.org for a long time. Does this affect #70 or #76?
@larsbergstrom
Copy link
Contributor

larsbergstrom commented Apr 4, 2016

@aneeshusa Thanks! Triaged :-)
@SimonSapin SimonSapin deleted the cargo branch Dec 20, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Linked issues

Successfully merging this pull request may close these issues.

None yet

5 participants
@Ms2ger @metajack @bors-servo @aneeshusa @larsbergstrom
You can’t perform that action at this time.