Skip to content

GH-50476: [R] empty_named_list() uses deprecated .Names in structure() #50485

Merged
thisisnic merged 1 commit into
apache:mainfrom
thisisnic:GH-50476-Namesnames
Jul 13, 2026
Merged

GH-50476: [R] empty_named_list() uses deprecated .Names in structure() #50485
thisisnic merged 1 commit into
apache:mainfrom
thisisnic:GH-50476-Namesnames

Conversation

@thisisnic

@thisisnic thisisnic commented Jul 13, 2026

Copy link
Copy Markdown
Member

Rationale for this change

Update .Names to names as CRAN check fails

What changes are included in this PR?

Update .Names to names as CRAN check fails

Are these changes tested?

CI

Are there any user-facing changes?

No

@thisisnic thisisnic requested a review from jonkeane as a code owner July 13, 2026 07:10
Copilot AI review requested due to automatic review settings July 13, 2026 07:10
@github-actions

Copy link
Copy Markdown

⚠️ GitHub issue #50476 has been automatically assigned in GitHub to PR creator.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the R helper empty_named_list() to avoid an R-devel deprecation warning by using the modern names argument in structure(), aligning Arrow’s R utilities with current R semantics and keeping CRAN/R-devel checks clean.

Changes:

  • Replace deprecated .Names usage with names in empty_named_list().

@thisisnic

Copy link
Copy Markdown
Member Author

@github-actions crossbow submit test-r-fedora-clang

@github-actions

Copy link
Copy Markdown

Revision: 8279391

Submitted crossbow builds: ursacomputing/crossbow @ actions-4a464d0e34

Task Status
test-r-fedora-clang GitHub Actions

@jonkeane jonkeane left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for this!

@github-actions github-actions Bot added awaiting merge Awaiting merge and removed awaiting committer review Awaiting committer review labels Jul 13, 2026
@thisisnic thisisnic merged commit 0a6de2d into apache:main Jul 13, 2026
34 checks passed
@thisisnic thisisnic removed the awaiting merge Awaiting merge label Jul 13, 2026
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.

3 participants