Skip to content

rxiv-maker 1.24.2 - #13

Merged
paxcalpt merged 1 commit into
mainfrom
rxiv-maker-1.24.2
Sep 15, 2026
Merged

paxcalpt merged 1 commit into
mainfrom
rxiv-maker-1.24.2

Conversation

@paxcalpt

Copy link
Copy Markdown
Contributor

Bump to 1.24.2: track-changes DOCX export keeps inline formatting (code font, subscripts, superscripts) inside marked paragraphs.

Copilot AI lite review requested due to automatic review settings September 15, 2026 12:40

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.

🟢 Approval recommended

No unresolved review comments remain.

Pull request overview

Updates the Homebrew formula for rxiv-maker to version 1.24.2.

Changes:

  • Updates the source archive URL.
  • Updates the SHA-256 checksum.
File summaries
File Description
Formula/rxiv-maker.rb Bumps the formula to version 1.24.2 and updates its checksum.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@claude

claude Bot commented Sep 15, 2026

Copy link
Copy Markdown

Review: rxiv-maker 1.24.2 bump

This is a clean, minimal version bump — exactly the shape expected for this repo (Homebrew formula tap for rxiv-maker).

Code quality / conventions

  • Diff is scoped correctly: only url and sha256 changed, both fields updated together (a common mistake is bumping the URL/version but forgetting the checksum — not the case here).
  • Both the old and new sha256 values are well-formed 64-character hex strings, consistent with a real SHA-256 digest.
  • No unrelated changes to desc, license, depends_on, install, caveats, or test blocks — appropriate since none of those should change for a patch release.

Potential issues

  • I couldn't fetch the tarball from files.pythonhosted.org in this sandbox to independently recompute the SHA-256 and confirm it matches 89d8b70dfb9f7f0aa480b79986df43efccf4c7ef761ee04062fd2c791cde342b. Homebrew's CI (brew test-bot / audit) will do this verification on merge, so this should be low risk, but worth confirming the checksum was copied from a trusted source (e.g., pypi.org's published hash for rxiv_maker-1.24.2.tar.gz) rather than hand-computed.
  • The PR description says this release "keeps inline formatting... inside marked paragraphs" for track-changes DOCX export — that's an upstream rxiv-maker changelog note, not something this formula change can be functionally tested against directly, so nothing to verify on the Homebrew side beyond the version/checksum match.

Test coverage

  • The existing test do block (rxiv --version / rxiv --help) is version-agnostic and will exercise the new install automatically — no changes needed there.

Security

  • No new dependencies or install-script changes; the install path (pip install . inside a venv) is unchanged. Since the sha256 pin is updated alongside the URL, the download remains integrity-checked.

Overall: straightforward, well-formed version bump. LGTM pending Homebrew CI's automated audit/checksum verification.

@paxcalpt
paxcalpt merged commit 3f9e96b into main Sep 15, 2026
2 checks passed
@paxcalpt
paxcalpt deleted the rxiv-maker-1.24.2 branch September 15, 2026 12:43
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