Skip to content

Use latest pandoc in render-readme workflow#1175

Open
nikosbosse wants to merge 1 commit intomainfrom
fix/render-readme-pandoc-version
Open

Use latest pandoc in render-readme workflow#1175
nikosbosse wants to merge 1 commit intomainfrom
fix/render-readme-pandoc-version

Conversation

@nikosbosse
Copy link
Copy Markdown
Collaborator

CLAUDE: ## Summary

  • Pin r-lib/actions/setup-pandoc@v2 to pandoc-version: 'latest' in the render-readme workflow.

Why

The default pandoc that ships with setup-pandoc@v2 is 3.1.11. That version renders consecutive HTML comments without separating blank lines, but pandoc 3.9.x (current Homebrew) inserts blank lines between them when emitting GFM. Concretely, the all-contributors block in README.Rmd:

```

```

renders to stacked comments under CI but to blank-line-separated comments under recent local pandoc. Anytime a locally-rendered README.md lands on main (e.g. commit d109d3f bundled one alongside an Rbuildignore edit), the next CI render produces a diff and opens an "Update README" PR (e.g. #1173, #1165) that just removes those blank lines. Bumping CI to latest ends the cycle for any reasonably current local install.

Test plan

  • Workflow YAML still parses (GitHub Actions check on this PR)
  • After merge: confirm next render-readme run on main does not produce a no-op "Update README" PR

The default pandoc shipped with setup-pandoc@v2 (3.1.11) renders
consecutive HTML comments without separating blank lines, while
recent pandoc (3.9.x) inserts them. This drift produced repeated
"Update README" PRs whenever a locally-rendered README.md (with
blank lines) was committed and CI then re-rendered without them.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@codecov
Copy link
Copy Markdown

codecov Bot commented May 1, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.96%. Comparing base (ea8b854) to head (6d79c4c).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1175   +/-   ##
=======================================
  Coverage   97.96%   97.96%           
=======================================
  Files          37       37           
  Lines        2012     2012           
=======================================
  Hits         1971     1971           
  Misses         41       41           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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