Skip to content

GH-50492: [R] Add release process skill#50499

Draft
thisisnic wants to merge 3 commits into
apache:mainfrom
thisisnic:r-release-skill
Draft

GH-50492: [R] Add release process skill#50499
thisisnic wants to merge 3 commits into
apache:mainfrom
thisisnic:r-release-skill

Conversation

@thisisnic

@thisisnic thisisnic commented Jul 13, 2026

Copy link
Copy Markdown
Member

Rationale for this change

Add a Claude Code skill to guide folks through the CRAN release process, and add .claude/ to the RAT excludes.

What changes are included in this PR?

  • New skill at .claude/skills/r-cran-release/SKILL.md with step-by-step CRAN release instructions
  • Add .claude/* to dev/release/rat_exclude_files.txt
  • Remove the "Prepare and Check Package That Will Be Released to CRAN" section from r/PACKAGING.md (covered by later steps)

Are these changes tested?

No code changes to test.

Are there any user-facing changes?

No.

thisisnic and others added 2 commits July 13, 2026 10:54
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Comment thread .claude/skills/r-cran-release/SKILL.md Outdated
@github-actions github-actions Bot added awaiting changes Awaiting changes and removed awaiting committer review Awaiting committer review labels Jul 20, 2026
Comment on lines +347 to +349
```r
pkgdown::build_news()
```

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Suggested change
```r
pkgdown::build_news()
```
```r
pkgdown::build_news("./r")

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Don't merge this like this, adding code blocks as comments gets weird so we should manually apply the change to make sure we get all the backticks

Comment on lines +351 to +353
Submit a PR to the `asf-site` branch of https://github.com/apache/arrow-site with contents of `arrow/r/docs/news/index.html` replacing `arrow-site/docs/r/news/index.html`.

Bump the version in `r/pkgdown/assets/versions.json` and update on the `asf-site` branch too.

@thisisnic thisisnic Jul 20, 2026

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Suggested change
Submit a PR to the `asf-site` branch of https://github.com/apache/arrow-site with contents of `arrow/r/docs/news/index.html` replacing `arrow-site/docs/r/news/index.html`.
Bump the version in `r/pkgdown/assets/versions.json` and update on the `asf-site` branch too.
Submit a PR to the `asf-site` branch of https://github.com/apache/arrow-site with contents of `arrow/r/docs/news/index.html` replacing `arrow-site/docs/r/news/index.html`.
```bash
cd ~/arrow-site
git fetch upstream
git checkout upstream/asf-site
git checkout -b "r-25.0.0"
cp ~/arrow/r/docs/news/index.html ./docs/r/news/index.html
git add docs/r/news/index.html
git commit -m "update R news page"
git push --set-upstream origin r-25.0.0

If necessary, bump the version in r/pkgdown/assets/versions.json and update on the asf-site branch too.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Don't merge this comment like this, adding code blocks as comments gets weird so we should manually apply the change to make sure we get all the backticks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant