-
-
Notifications
You must be signed in to change notification settings - Fork 580
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
[docs] Improve formatting, fix typos, remove duplicates #4726
Conversation
He's back, yay! |
Literally every time I read the docs, I find something useful that I can apply to my configuration and make my workflow easier. Great work! |
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.
Great stuff, thanks so much for careful reading and continuing contributions!
A couple of misunderstandings, etc. Would be happy to have suggestions about how to make these misunderstandings less likely.
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.
Thanks for all the work on this. The goal with "Prerelease tasks" is to integrate the second set (6 steps) into the first set (bullets). You were right that they shouldn't both be there, but they need to be merged.
I understand that it needs to be merged, but I don't know how. I think I'll leave it to @mattstein, so that "Creating a Release" and "Pushing Docker Images with the GitHub Actions Workflow" don't look nearly the same. |
I can merge those sections. |
Co-authored-by: Randy Fay <randy@randyfay.com>
Rebased and merged prerelease tasks into one section. |
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.
The only thing I don’t like about this PR is how much of it I missed in the first place. Thanks @stasadev! Soft unrelated wording suggestion but otherwise it looks good to me.
Co-authored-by: Matt Stein <m@ttste.in>
No description provided.