Skip to content

Default review mode true; CLI injects publish form#105

Merged
PatrikBak merged 1 commit into
mainfrom
patrik/review-mode-default-true
May 16, 2026
Merged

Default review mode true; CLI injects publish form#105
PatrikBak merged 1 commit into
mainfrom
patrik/review-mode-default-true

Conversation

@PatrikBak
Copy link
Copy Markdown
Owner

Summary

  • Inverts the \ReviewMode default in _template.tex from false to true, so manual pdfcsplain compiles always render the inline-solutions review form without per-file opt-in.
  • BuildCommand.CompileTexFile now defines \PUBLISH on the pdfcsplain command line before \input; the template flips \ReviewMode off when it sees the sentinel, so the CLI is the single gate that produces the publish PDF.
  • Removes the stale \ReviewModetrue line from angle-basics-1.sk.tex — the override that accidentally shipped to main. Grep confirmed no other handouts touch the macro.

`\ReviewModetrue` was an opt-in toggle authors flipped on while reviewing
a draft locally. The hazard: the toggle lived in the .tex source itself,
so a forgotten line on a finished handout shipped the review form to the
website (it just did, on angle-basics-1.sk.tex).

The local workflow is always review; only the Handouts CLI produces the
publish form. So invert the default: the template now starts `\ReviewMode`
true, and the CLI defines a `\PUBLISH` sentinel on the pdfcsplain command
line before \input — the template flips itself off when it sees the
sentinel. Manual pdfcsplain compiles keep producing the inline-solutions
PDF; the CLI is the single gate that produces the publish PDF.
@vercel
Copy link
Copy Markdown

vercel Bot commented May 16, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
math-comps Ready Ready Preview, Comment May 16, 2026 11:32pm

@PatrikBak PatrikBak enabled auto-merge May 16, 2026 23:31
@PatrikBak PatrikBak merged commit 518892f into main May 16, 2026
4 checks passed
@PatrikBak PatrikBak deleted the patrik/review-mode-default-true branch May 16, 2026 23:35
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.

1 participant