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

Fix buildbot step description for shell commands #472

Merged
merged 1 commit into from Aug 23, 2016

Conversation

@UK992
Copy link
Contributor

UK992 commented Aug 23, 2016

This should fix description for shell commands where after #438 shows only "ran", instead of command.
r? @aneeshusa


This change is Reviewable

@UK992 UK992 force-pushed the UK992:buildbot-steps branch from cf85b91 to f440438 Aug 23, 2016
@aneeshusa
Copy link
Member

aneeshusa commented Aug 23, 2016

@bors-servo r+

Thanks for catching that, I missed that during review!

@bors-servo
Copy link
Contributor

bors-servo commented Aug 23, 2016

📌 Commit f440438 has been approved by aneeshusa

@bors-servo
Copy link
Contributor

bors-servo commented Aug 23, 2016

Testing commit f440438 with merge 0b3c861...

bors-servo added a commit that referenced this pull request Aug 23, 2016
Fix buildbot step description for shell commands

This should fix description for shell commands where after #438 shows only "ran", instead of command.
r? @aneeshusa

<!-- 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/472)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Aug 23, 2016

☀️ Test successful - status-travis

@bors-servo bors-servo merged commit f440438 into servo:master Aug 23, 2016
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
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

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