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

README Clarifications #585

Merged
merged 3 commits into from
May 29, 2016
Merged

README Clarifications #585

merged 3 commits into from
May 29, 2016

Conversation

nathenharvey
Copy link

Some minor copy and formatting changes.

Clarify instructions for building a sample plan by creating a key hab origin key generate myorigin and adding that key as an environment variable, export HAB_ORIGIN=myorigin.

@thesentinels
Copy link
Contributor

By analyzing the blame information on this pull request, we identified @smith, @adamhjk, @fnichol, @irvingpop and @metadave to be potential reviewers

@thesentinels
Copy link
Contributor

☔ The latest upstream changes (presumably be0e518) made this pull request unmergeable. Please resolve the merge conflicts.

@@ -220,9 +221,12 @@ Each commit must include a DCO which looks like this

`Signed-off-by: Joe Smith <[email protected]>`

The project requires that the name used is your real name. No contributions utilizing pseudonyms will be accepted nor will anonymous contributions.
The project requires that the name used is your real name. No contributions utilizing pseudonyms
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This last sentence should really be written "Neither anonymous contributors nor those utilizing pseudonyms will be accepted."

@juliandunn
Copy link
Contributor

@thesentinels r+

@thesentinels
Copy link
Contributor

📌 Commit 559858a has been approved by juliandunn

thesentinels pushed a commit that referenced this pull request May 29, 2016
Signed-off-by: Nathen Harvey <[email protected]>

Pull request: #585
Approved by: juliandunn
thesentinels pushed a commit that referenced this pull request May 29, 2016
Signed-off-by: Nathen Harvey <[email protected]>

Pull request: #585
Approved by: juliandunn
thesentinels pushed a commit that referenced this pull request May 29, 2016
Signed-off-by: Nathen Harvey <[email protected]>

Pull request: #585
Approved by: juliandunn
@thesentinels
Copy link
Contributor

⌛ Testing commit 559858a with merge fc8ff18...

@thesentinels
Copy link
Contributor

☀️ Test successful - travis

@thesentinels thesentinels merged commit 559858a into habitat-sh:master May 29, 2016
@nathenharvey nathenharvey deleted the nathen/readme branch May 30, 2016 13:02
jtimberman pushed a commit that referenced this pull request Jun 12, 2016
Signed-off-by: Nathen Harvey <[email protected]>

Pull request: #585
Approved by: juliandunn
jtimberman pushed a commit that referenced this pull request Jun 12, 2016
Signed-off-by: Nathen Harvey <[email protected]>

Pull request: #585
Approved by: juliandunn
jtimberman pushed a commit that referenced this pull request Jun 12, 2016
Signed-off-by: Nathen Harvey <[email protected]>

Pull request: #585
Approved by: juliandunn
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.

3 participants