-
Notifications
You must be signed in to change notification settings - Fork 8.1k
Added Configure GitHub Actions step to the Python guide #22676
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
Added Configure GitHub Actions step to the Python guide #22676
Conversation
❌ Deploy Preview for docsdocker failed. Why did it fail? →
|
rw4lll
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.
Looks good to me. I think it's ready to be merged.
craig-osterhout
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.
Thanks @igor-alexandrov.
Only a couple small issues I think it's good to go.
- At the end of
containerize.md, the next steps section needs to be updated to reflect the different ordering of the topics. It should say ci/cd is next rather than develop. - At the end of
develop.md, the next steps section needs to be updated to reflect the different ordering of the topics. It should say test deployment is next rather than ci/cd.
|
@craig-osterhout thank you! I updated both parts of the guide. |
craig-osterhout
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.
Thank you. LGTM
Description
Replaced "Configure CI/CD" step of the Python guide with "Configure GitHub Actions" step to have it similar structure as the Rails guide.
Related issues or tickets
#22114
Reviews