-
Notifications
You must be signed in to change notification settings - Fork 12
Update content on deployment to cover pipelines #2
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
Conversation
IEvangelist
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I feel like this is a good start to the draft, but seems like there's a lot of missing details and not enough prose. Some of the reasonings are missing, like "why should I care about pipelines?"
|
Hey @captainsafia - looks like we need to address the merge conflict too. 😬 |
58b9497 to
f8b58cf
Compare
|
@IEvangelist I ended up redoing a bunch of this so not all the feedback from above still applies. I fixed the stuff that was still applicable. Taking this out of draft for official review. |
Update the docs to cover pipelines as the core concept for publishing + deploying.