Skip to content

Fix duplicate version in older releases table#1972

Merged
cderv merged 3 commits intomainfrom
fix/duplicate-older-release
Mar 24, 2026
Merged

Fix duplicate version in older releases table#1972
cderv merged 3 commits intomainfrom
fix/duplicate-older-release

Conversation

@cderv
Copy link
Copy Markdown
Member

@cderv cderv commented Mar 24, 2026

The "Older Releases" table on the download page shows version 1.8.27 twice.

Root Cause

tools/release-notes.R uses cat(..., append = TRUE) to add entries to
docs/download/_download-older.yml. If the script runs twice (e.g. re-running
the release checklist), it blindly appends a duplicate with no guard.

Fix

  • Remove the duplicate 1.8.27 entry from _download-older.yml
  • Replace the cat(append) approach with yaml12-based upsert: the script now
    parses the existing YAML, checks if the version is already present, and
    either updates it or appends a new entry
  • Add yaml12 as a project dependency (DESCRIPTION + renv.lock)

cderv added 3 commits March 24, 2026 08:16
The version 1.8.27 entry was appended twice to _download-older.yml,
causing it to appear as a duplicate row on the download page.
yaml12 will be used in release-notes.R for idempotent updates to
_download-older.yml, preventing duplicate entries.
Replace blind cat(append=TRUE) with yaml12-based read/check/write.
The script now parses the existing YAML, checks if the version is
already present, and either updates it or appends a new entry.
This prevents duplicate entries if the script runs twice.
@github-actions github-actions Bot temporarily deployed to pull request March 24, 2026 15:33 Inactive
@github-actions
Copy link
Copy Markdown
Contributor

📝 Preview Deployment

🔍 Full site preview: https://deploy-preview-1972.quarto.org

@cderv cderv merged commit afe4421 into main Mar 24, 2026
4 checks passed
@cderv cderv deleted the fix/duplicate-older-release branch March 24, 2026 16:15
github-actions Bot pushed a commit that referenced this pull request Mar 24, 2026
* Fix duplicate 1.8.27 entry in older releases listing

The version 1.8.27 entry was appended twice to _download-older.yml,
causing it to appear as a duplicate row on the download page.

* Add yaml12 dependency for structured YAML handling

yaml12 will be used in release-notes.R for idempotent updates to
_download-older.yml, preventing duplicate entries.

* Use yaml12 for idempotent upsert in release-notes.R

Replace blind cat(append=TRUE) with yaml12-based read/check/write.
The script now parses the existing YAML, checks if the version is
already present, and either updates it or appends a new entry.
This prevents duplicate entries if the script runs twice.

(cherry picked from commit afe4421)
@github-actions
Copy link
Copy Markdown
Contributor

Successfully created backport PR for prerelease:

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