Skip to content

Conversation

@oesteban
Copy link
Member

@oesteban oesteban commented Oct 8, 2025

Summary

  • generate tutorial notebooks from markdown files during the deploy workflow
  • automatically commit updated notebooks back to main when the workflow runs on push to main

Testing

  • not run

https://chatgpt.com/codex/tasks/task_e_68e5f29a92148330b752e0423a9ea1eb

Resolves: #27. The idea of sphinx-exercises is abandoned because of theme restrictions and a repo that doesn't look very active

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting

Comment on lines +74 to +78
git config user.name "github-actions[bot]"
git config user.email "41898282+github-actions[bot]@users.noreply.github.com"
git add docs/notebook/*.ipynb
git commit -m "Update tutorial notebooks"
git push

Choose a reason for hiding this comment

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

P1 Badge Ensure push step handles remote updates

The new auto-commit step pushes notebook changes straight to main without first rebasing or fetching. If someone else pushes to main while this workflow run is still executing, git push will fail with a non-fast-forward error and the entire deploy job will be marked failed even though notebook generation succeeded. Consider pulling with rebase (or skipping the commit) when the local branch is behind to avoid sporadic deployment failures.

Useful? React with 👍 / 👎.

@oesteban oesteban merged commit 44c255a into main Oct 8, 2025
1 check passed
@oesteban oesteban deleted the codex/generate-jupyter-notebooks-from-tutorial branch October 8, 2025 05:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Solution to the exercises

2 participants