Force LF line endings for R files in .gitattributes - #604
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates project configuration files by adding .positai and .claude to .Rbuildignore, enforcing LF line endings for R files in .gitattributes to support the air formatter, and adding .positai to .gitignore. A review comment correctly points out that .claude should also be added to .gitignore for consistency with the build ignore list and to prevent accidental commits.
| docs | ||
| *.tar.gz | ||
| report.Rcheck | ||
| .positai |
There was a problem hiding this comment.
The file .claude was added to .Rbuildignore but is missing from .gitignore. It should be added here as well to ensure it is not accidentally committed to the repository, maintaining consistency with the addition of .positai.
.positai
.claude
There was a problem hiding this comment.
Pull request overview
This PR updates repository configuration to ensure consistent LF line endings for R source files (to align with the air formatter expectations) and to exclude local tool configuration files from package builds and/or git tracking.
Changes:
- Enforce LF line endings for
*.Rvia.gitattributes. - Exclude
.positaiand.claudefrom R package builds via.Rbuildignore. - Ignore
.positaiin git via.gitignore.
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
.Rbuildignore |
Adds build-exclusion patterns for .positai and .claude. |
.gitignore |
Prevents committing .positai to the repository. |
.gitattributes |
Forces LF EOL for *.R files to avoid CRLF-related formatter issues. |
No description provided.