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.

Sign up for GitHub

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

Jump to bottom

docs #36

Merged
merged 14 commits into from
Jul 17, 2018
Merged

docs #36

merged 14 commits into from
Jul 17, 2018

Conversation

ryanking
Copy link
Contributor

@ryanking ryanking commented Jul 16, 2018 •
edited
Loading

Fixes #46

@ryanking ryanking requested a review from a team as a code owner July 16, 2018 22:56
@ryanking ryanking changed the title Ryanking/docs docs Jul 16, 2018
edulop91
edulop91 reviewed Jul 16, 2018
View reviewed changes
README.md Outdated

### Transparency

Fogg tries to stay out of your way– it will do its work by generating Terraform and Make files, and then it step aside for you to manage your infrastructure. Everything that could effect your infrastructure is right there in your resposityr for you to read and understand.
Copy link
Contributor

Choose a reason for hiding this comment

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

resposityr -> repository

@austinylin
Copy link
Contributor

Do we want a section on how to contribute?

@davidrecordon do we need a CLA?

@austinylin
Copy link
Contributor

We may also want to call out how to get started.

kislyuk
kislyuk reviewed Jul 17, 2018
View reviewed changes
README.md Outdated

It makes life easy for folks working with cloud infrastructure. We've been using fogg and its predecessor internally at CZI for ~10 months. It has made it possible for many developers without terraform experience to safely roll new infrastructure with less stress and higher quality.

"I hope one day you might consider open sourcing `fogg`, i really love it. This would of saved me so much time in the past." - @lenn0x
Copy link

Choose a reason for hiding this comment

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

"would have"

@ryanking
Copy link
Contributor Author

@austinylin yeah I added a section about how to contributed. In the past the word has been we don't need a CLA.

I am going to add a full-on tutorial outside this README when I get a chance.

@ryanking ryanking self-assigned this Jul 17, 2018
@czimergebot czimergebot merged commit 06881b4 into master Jul 17, 2018
@vincenthsh vincenthsh mentioned this pull request Feb 23, 2023
Closed
@vincenthsh vincenthsh mentioned this pull request Mar 15, 2023
Closed
@vincenthsh vincenthsh mentioned this pull request Sep 13, 2023
Closed
@vincenthsh vincenthsh mentioned this pull request Apr 24, 2024
Closed
@vincenthsh vincenthsh mentioned this pull request Nov 12, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kislyuk kislyuk kislyuk left review comments

@chrisgoffinet chrisgoffinet chrisgoffinet approved these changes

@edulop91 edulop91 edulop91 approved these changes

Assignees

@ryanking ryanking

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@ryanking @austinylin @kislyuk @edulop91 @chrisgoffinet @czimergebot