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

Use best practices in artifactory-pro run hook #995

Merged

Conversation

schisamo
Copy link
Contributor

These updates are based on feedback from @jamesc in #989.

Signed-off-by: Seth Chisamore [email protected]

/cc @smith

These updates are based on feedback from @jamesc in habitat-sh#989.

Signed-off-by: Seth Chisamore <[email protected]>
@thesentinels
Copy link
Contributor

By analyzing the blame information on this pull request, we identified @adamhjk to be a potential reviewer

@reset
Copy link
Collaborator

reset commented Jun 27, 2016

@thesentinels r+

Thank you!

@smith
Copy link
Contributor

smith commented Jun 27, 2016

@thesentinels r+

@thesentinels
Copy link
Contributor

📌 Commit bb3152b has been approved by smith

@thesentinels
Copy link
Contributor

⌛ Testing commit bb3152b with merge 53eaf2b...

thesentinels added a commit that referenced this pull request Jun 27, 2016
Use best practices in artifactory-pro run hook

These updates are based on feedback from @jamesc in #989.

Signed-off-by: Seth Chisamore <[email protected]>

/cc @smith
@thesentinels
Copy link
Contributor

☀️ Test successful - travis

@thesentinels thesentinels merged commit bb3152b into habitat-sh:master Jun 27, 2016
@smith
Copy link
Contributor

smith commented Jun 28, 2016

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