Skip to content

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

igor-alexandrov
Copy link
Contributor

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

  • Technical review
  • Editorial review
  • Product review

Copy link

netlify bot commented May 21, 2025

Deploy Preview for docsdocker failed. Why did it fail? →

Name Link
🔨 Latest commit f781025
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/6841450943f63a0008a94d82

Copy link
Contributor

@rw4lll rw4lll left a 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 craig-osterhout requested a review from a team June 3, 2025 17:03
Copy link
Contributor

@craig-osterhout craig-osterhout left a 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.

@igor-alexandrov
Copy link
Contributor Author

@craig-osterhout thank you! I updated both parts of the guide.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants