Skip to content

Upgrade sphinx-exercise to 1.2.2 - #317

Closed
mmcky wants to merge 1 commit into
mainfrom
maint/sphinx-exercise-1.2.2
Closed

mmcky wants to merge 1 commit into
mainfrom
maint/sphinx-exercise-1.2.2

Conversation

@mmcky

@mmcky mmcky commented Nov 10, 2026 •

Copy link
Copy Markdown
Contributor

Upgrades sphinx-exercise from 1.2.1 to 1.2.2 in environment.yml, the file that CI, cache and publish build with.

sphinx-exercise 1.2.1 wrote a captioned figure inside an exercise or solution box as a floating \begin{figure}[htbp], which LaTeX rejects there with Not in outer par mode. 1.2.2 writes it as [H], as Sphinx does inside its own admonitions; see executablebooks/sphinx-exercise#95.

Why now: QuantEcon/lecture-python.myst#1121 restores nine figure captions inside solutions, which reach this repository through the translation sync. This repository builds HTML only (the PDF steps in ci.yml and publish.yml are commented out), so 1.2.1 would not fail on them. This bump keeps the pin in step with the English source, and the two PRs can merge in either order.

The release has the same dependencies as 1.2.1, and HTML output is unchanged. environment-cn.yml (pinned at 1.2.0) is used only by the inactive .github/workflows/_future/ci-cn.yml, so this PR leaves it alone.

Part of QuantEcon/workspace-lectures#70.

🤖 Generated with Claude Code

https://claude.ai/code/session_01WT8bMZoSvoQTbzkdmZiMnd

sphinx-exercise 1.2.2 writes a captioned figure inside an exercise or solution box as [H], so the LaTeX build no longer stops with "Not in outer par mode".
This lands before QuantEcon/lecture-python.myst#1121, whose restored captions in solutions reach this repository through the translation sync.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01WT8bMZoSvoQTbzkdmZiMnd
Copilot AI balanced review requested due to automatic review settings November 10, 2026 05:36
@netlify

netlify Bot commented Nov 10, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for astonishing-narwhal-a8fc64 ready!

Name Link
🔨 Latest commit a35649a
🔍 Latest deploy log https://app.netlify.com/projects/astonishing-narwhal-a8fc64/deploys/6af2ad4fd5fce00b3fb3de97
😎 Deploy Preview https://deploy-preview-317--astonishing-narwhal-a8fc64.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Note

Copilot was unable to complete this review at the selected tier due to model policy or plan restrictions.

🟢 Approval recommended

The change is a straightforward dependency pin update to sphinx-exercise 1.2.2 in environment.yml with no review evidence of unresolved issues or required follow-ups.

No open issues 🎉

What changed in this PR

Upgrades the CI/cache/publish build environment to pick up the sphinx-exercise LaTeX fix that prevents PDF build failures when captioned figures appear inside exercise/solution boxes.

Changes:

  • Bump sphinx-exercise from 1.2.1 to 1.2.2 in environment.yml.
File Description
environment.yml Updates the pinned sphinx-exercise version to 1.2.2 for the main build environment used by CI/cache/publish.

Review effort: Balanced

@github-actions

Copy link
Copy Markdown

@mmcky

mmcky commented Nov 10, 2026

Copy link
Copy Markdown
Contributor Author

Closing as not needed: this repository builds HTML only (its PDF steps are commented out), and sphinx-exercise 1.2.2 changes only LaTeX output. Dependabot's weekly conda update will bring the pin in step.

@mmcky mmcky closed this Nov 10, 2026
@mmcky
mmcky deleted the maint/sphinx-exercise-1.2.2 branch November 10, 2026 05:53

This branch was successfully deployed

1 active deployment
pull request — a35649a3 Deployed Nov 10, 2026 by github-actions[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants