Skip to content

Conversation

davidorme
Copy link
Collaborator

@davidorme davidorme commented Oct 6, 2025

This PR is simply to update the pre-commit settings to try and fix issues reported by @arne-exe in #123.

  • It also removes a linting check for code complexity, which is no longer a standard part of the linter checking - and probably isn't a priority for R scripts (as opposed to package and function level complexity).
  • It fixes a new markdown issue revealed by pre-commit run -a

@davidorme davidorme linked an issue Oct 6, 2025 that may be closed by this pull request
Copy link
Collaborator

@jacobcook1995 jacobcook1995 left a comment

Choose a reason for hiding this comment

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

LGTM!

@davidorme davidorme merged commit 5776a40 into main Oct 6, 2025
2 checks passed
@davidorme davidorme deleted the 124-issues-with-pre-commit-setup branch October 6, 2025 12:10
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.

Issues with pre-commit setup
2 participants