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

Add ssh_{public,private}_key_ssm_path outputs #5

Merged
merged 2 commits into from
Sep 18, 2019

Conversation

rsrchboy
Copy link
Contributor

...as these could be construed as useful :)

@osterman osterman requested review from maximmi and aknysh February 18, 2019 05:06
Copy link
Member

@osterman osterman left a comment

Choose a reason for hiding this comment

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

Just one thing... The README.md is generated from the README.yaml. Make all your changes to the README.yaml, then rebuild.

Please rebuild the README.md by running the follow commands:

make init
make readme/deps
make readme

There might be a lot of changes in the README.md as a result. That's okay!

Sorry for the inconvenience!

@osterman
Copy link
Member

Btw, you should totally join our community slack team! This is another great place to ask questions and get quickly unblocked.

https://slack.cloudposse.com

Slack Community

output.tf Outdated Show resolved Hide resolved
output.tf Outdated Show resolved Hide resolved
Copy link
Member

@aknysh aknysh left a comment

Choose a reason for hiding this comment

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

@rsrchboy thanks!
please see comments and rebuild README

@rsrchboy
Copy link
Contributor Author

Oh, nice -- I didn't realize the README is built that way. Done.

@rsrchboy
Copy link
Contributor Author

@osterman were the changes sufficient?

@osterman
Copy link
Member

Oops! Looks like we missed this. Yes looks good. Can you rebase or fix the merge conflicts?

@rsrchboy
Copy link
Contributor Author

@osterman done!

@aknysh
Copy link
Member

aknysh commented Sep 18, 2019

/codefresh run test

@aknysh aknysh merged commit 45bd961 into cloudposse:master Sep 18, 2019
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