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

Give myself permission to `@bors-servo try`. #162

Merged
merged 1 commit into from Nov 15, 2015
Merged

Give myself permission to `@bors-servo try`. #162

merged 1 commit into from Nov 15, 2015
+1 −0

Conversation

@notriddle
Copy link
Contributor

notriddle commented Nov 14, 2015

@jdm's idea

Review on Reviewable
@notriddle
Give myself permission to `@bors-servo try`.
Loading status checks…
dd98a91
@Manishearth
Copy link
Member

Manishearth commented Nov 14, 2015

Seconded, though I don't know what our governance structure is around granting try, so I'll wait for @larsbergstrom's seal of approval
@larsbergstrom
Copy link
Contributor

larsbergstrom commented Nov 15, 2015

@bors-servo r+

Recommendation from a reviewer seems more than adequate, much less two of them! CC @metajack in case he wants to discuss more at the next meeting.
@bors-servo
Copy link
Contributor

bors-servo commented Nov 15, 2015

📌 Commit dd98a91 has been approved by larsbergstrom
@bors-servo
Copy link
Contributor

bors-servo commented Nov 15, 2015

⌛ Testing commit dd98a91 with merge 53897a9...
bors-servo added a commit that referenced this pull request Nov 15, 2015
@bors-servo
Auto merge of #162 - notriddle:master, r=larsbergstrom
Loading status checks…
53897a9 
Give myself permission to `@bors-servo try`.

[@jdm's idea](servo/servo#8496 (comment))

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/saltfs/162)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Nov 15, 2015

💔 Test failed - travis
@larsbergstrom
Copy link
Contributor

larsbergstrom commented Nov 15, 2015 

120.18s$ .travis/install_salt "${TRAVIS_OS_NAME}"

.travis/install_salt: installing salt for Linux

  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current

                                 Dload  Upload   Total   Spent    Left  Speed

  0     0    0     0    0     0      0      0 --:--:--  0:02:00 --:--:--     0

curl: (35) Unknown SSL protocol error in connection to repo.saltstack.com:443 

gpg: no valid OpenPGP data found.

The command ".travis/install_salt "${TRAVIS_OS_NAME}"" failed and exited with 2 during .

Hrm, this appears to be an infrastructure error. Sorry for the delay while we sort this out, @notriddle!
@Manishearth
Copy link
Member

Manishearth commented Nov 15, 2015

@bors-servo retry
@bors-servo
Copy link
Contributor

bors-servo commented Nov 15, 2015

⌛ Testing commit dd98a91 with merge dd89604...
bors-servo added a commit that referenced this pull request Nov 15, 2015
@bors-servo
Auto merge of #162 - notriddle:master, r=larsbergstrom
Loading status checks…
dd89604 
Give myself permission to `@bors-servo try`.

[@jdm's idea](servo/servo#8496 (comment))

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/saltfs/162)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Nov 15, 2015

☀️ Test successful - travis
@bors-servo bors-servo merged commit dd98a91 into servo:master Nov 15, 2015
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
@Manishearth
Copy link
Member

Manishearth commented Nov 15, 2015

Deployed. Enjoy your new try privileges!
@notriddle
Copy link
Contributor Author

notriddle commented Nov 15, 2015

Deployed. Enjoy your new try privileges!

I'll try.

On Sat, Nov 14, 2015, 23:02 Manish Goregaokar notifications@github.com
wrote:

Deployed. Enjoy your new try privileges!

—
Reply to this email directly or view it on GitHub
#162 (comment).
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

4 participants
@notriddle @Manishearth @larsbergstrom @bors-servo
You can’t perform that action at this time.