Skip to content

Conversation

@aeschli
Copy link
Contributor

@aeschli aeschli commented Jan 22, 2026

fixes #284740

Copilot AI review requested due to automatic review settings January 22, 2026 14:43
@aeschli aeschli enabled auto-merge (squash) January 22, 2026 14:43
@aeschli aeschli self-assigned this Jan 22, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request aims to change the default R file extension from lowercase .r to uppercase .R to align with R source code conventions, R IDEs (RStudio, Positron), and the Tidyverse Style Guide.

Changes:

  • Updates the R language extension configuration to use capitalized file extensions (.R, .Rhistory, .Rprofile)

Comment on lines +20 to +22
".R",
".Rhistory",
".Rprofile",
Copy link

Copilot AI Jan 22, 2026

Choose a reason for hiding this comment

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

The change completely removes support for lowercase file extensions (.r, .rhistory, .rprofile). While .R should be the default/preferred extension (first in the list), the lowercase variants should still be supported for backward compatibility. Many existing R files use lowercase extensions and would lose language support with this change. The extensions array should include both uppercase and lowercase versions, with .R listed first to make it the default.

Suggested change
".R",
".Rhistory",
".Rprofile",
".R",
".r",
".Rhistory",
".rhistory",
".Rprofile",
".rprofile",

Copilot uses AI. Check for mistakes.
@vs-code-engineering vs-code-engineering bot added this to the January 2026 milestone Jan 22, 2026
@aeschli aeschli merged commit 41aebd8 into main Jan 22, 2026
22 checks passed
@aeschli aeschli deleted the aeschli/ruling-mongoose-903 branch January 22, 2026 21:09
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.

Use .R (not .r) as default R file extension

3 participants