GH-51152: [R] test-r-linux-as-cran nightly fails with NOTE about non-standard top-level file jarl.toml - #51154
Conversation
|
@github-actions crossbow submit test-r-linux-as-cran |
|
|
There was a problem hiding this comment.
🟡 Changes recommended
The PR description still contains the unedited template text (including breaking/critical fix boilerplate) and should be updated/cleaned up before merge.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
Pull request overview
This PR addresses an R CMD check NOTE in the CRAN-style CI job caused by r/jarl.toml being included in the built R package tarball as a non-standard top-level file.
Changes:
- Add
^jarl\.toml$tor/.Rbuildignoresojarl.tomlis excluded from the R package build. - Remove the now-unused
r/.lintrconfiguration file. - Remove the corresponding
r/.lintrexclusion entry from the RAT release exclude list.
File summaries
| File | Description |
|---|---|
| r/.Rbuildignore | Adds an ignore rule for jarl.toml to prevent CRAN NOTE. |
| r/.lintr | Deletes the lintr configuration file (no longer referenced). |
| dev/release/rat_exclude_files.txt | Removes obsolete exclusion for a deleted file. |
Review details
- Files reviewed: 3/3 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.
|
Revision: b55a691 Submitted crossbow builds: ursacomputing/crossbow @ actions-ee963f0f15
|
Rationale for this change
The test-r-linux-as-cran nightly job has been failing since 2026-08-25 because
r/jarl.toml(added in #50851) ends up in the package tarball and R CMD check flags it as a non-standard top-level file.What changes are included in this PR?
Adds
jarl.tomltor/.Rbuildignore, and removes the now-unusedr/.lintrconfig along with its.Rbuildignoreand RAT exclude entries.Are these changes tested?
Checked locally that the
.Rbuildignorepattern matchesjarl.tomlwithout excluding anything else. Thetest-r-linux-as-crancrossbow job will confirm the NOTE is gone.Are there any user-facing changes?
No.