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

Remove "sidecar" jargon and refer to it as the supervisor's HTTP API. #570

Merged
merged 1 commit into from
May 27, 2016

Conversation

juliandunn
Copy link
Contributor

A few other copy edits here and there.

@thesentinels
Copy link
Contributor

By analyzing the blame information on this pull request, we identified @cwebberOps and @ryankeairns to be potential reviewers

@reset
Copy link
Collaborator

reset commented May 26, 2016

@thesentinels r+

gif-keyboard-12646435771099346400

@thesentinels
Copy link
Contributor

📌 Commit 065d244 has been approved by reset

@davidwrede
Copy link

You took care of one of my Trello cards. :) Thanks @juliandunn!

@bookshelfdave
Copy link
Contributor

btw, hab-sup still has --listen-sidecar as a param

On May 26, 2016, at 8:20 PM, David Wrede [email protected] wrote:

You took care of one of my Trello cards. :) Thanks @juliandunn!


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@thesentinels
Copy link
Contributor

⌛ Testing commit 065d244 with merge c8901c1...

thesentinels pushed a commit that referenced this pull request May 27, 2016
Signed-off-by: Julian C. Dunn <[email protected]>

Pull request: #570
Approved by: reset
@thesentinels
Copy link
Contributor

☀️ Test successful - travis

@thesentinels thesentinels merged commit 065d244 into master May 27, 2016
@fnichol fnichol deleted the jcd/remove-sidecar branch May 27, 2016 04:01
jtimberman pushed a commit that referenced this pull request Jun 12, 2016
Signed-off-by: Julian C. Dunn <[email protected]>

Pull request: #570
Approved by: reset
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants