Skip to content

fix: respect repo linter configurations#26

Merged
falgreen merged 3 commits intomainfrom
fix/use-local-linter-configurations-if-they-already-exists
May 20, 2025
Merged

fix: respect repo linter configurations#26
falgreen merged 3 commits intomainfrom
fix/use-local-linter-configurations-if-they-already-exists

Conversation

@akselthomsen
Copy link
Copy Markdown
Member

Only download general linter specifications if they do not already exist in the repo calling the workflow.

Signed-off-by: Aksel Thomsen <[email protected]>
@akselthomsen akselthomsen self-assigned this May 20, 2025
@akselthomsen akselthomsen added the enhancement New feature or request label May 20, 2025
@akselthomsen akselthomsen linked an issue May 20, 2025 that may be closed by this pull request
Signed-off-by: Aksel Thomsen <[email protected]>
Signed-off-by: Aksel Thomsen <[email protected]>
@github-actions
Copy link
Copy Markdown

Github pages

Review the pkgdown webpage for the PR here

@github-actions
Copy link
Copy Markdown

Code coverage

Name Coverage (%)
r.workflows 100
R/hello.R 100

@akselthomsen akselthomsen marked this pull request as ready for review May 20, 2025 10:29
@akselthomsen akselthomsen requested a review from falgreen May 20, 2025 10:29
Copy link
Copy Markdown
Contributor

@falgreen falgreen left a comment

Choose a reason for hiding this comment

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

This is so nice, it will make the workflows really easy to configure.

@falgreen falgreen merged commit f5fe3be into main May 20, 2025
14 checks passed
@falgreen falgreen deleted the fix/use-local-linter-configurations-if-they-already-exists branch May 20, 2025 10:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Use local linter configurations if they already exists

2 participants