Skip to content

Add {lintr} pre-commit hook #1055

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
lorenzwalthert opened this issue Nov 8, 2022 · 1 comment · Fixed by #1064
Closed

Add {lintr} pre-commit hook #1055

lorenzwalthert opened this issue Nov 8, 2022 · 1 comment · Fixed by #1064
Assignees
@IndrajeetPatil
Labels
Complexity: Low Priority: Medium Type: Infrastructure

Comments

@lorenzwalthert
Copy link
Collaborator

with the config developed here. I am not sure how to store this in a .lintr file. Any idea @IndrajeetPatil?
@IndrajeetPatil
Copy link
Collaborator

I will take care of this sometime this week.

@IndrajeetPatil IndrajeetPatil self-assigned this Nov 8, 2022
IndrajeetPatil added a commit that referenced this issue Nov 10, 2022
@IndrajeetPatil
Add lintr config file
c7f6f7b 
Related to #1055
@IndrajeetPatil IndrajeetPatil mentioned this issue Nov 10, 2022
Merged
IndrajeetPatil added a commit that referenced this issue Nov 15, 2022
@IndrajeetPatil
Format precommit YAML files
969b378 
Need to handle this in a separate PR before I can work on #1055.
@IndrajeetPatil IndrajeetPatil mentioned this issue Nov 15, 2022
Merged
IndrajeetPatil added a commit that referenced this issue Nov 21, 2022
@IndrajeetPatil
Add {lintr} precommit hook
34e63c4 
Closes #1055
@IndrajeetPatil IndrajeetPatil mentioned this issue Nov 21, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@IndrajeetPatil IndrajeetPatil

Labels
Complexity: Low Priority: Medium Type: Infrastructure
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add {lintr} precommit hook and remove existing lints r-lib/styler
2 participants
@lorenzwalthert @IndrajeetPatil