-
Notifications
You must be signed in to change notification settings - Fork 0
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.
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
Add permissions check #30
Add permissions check #30
Conversation
.pre-commit-config.yaml
Outdated
#- repo: https://github.com/adrienverge/yamllint | ||
# rev: v1.32.0 | ||
# hooks: | ||
# - id: yamllint |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Temporarily commented out because I made the branch name too long
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[Question, non-blocking] Out of curiosity, how does the branch name affect the yamllint
hook?
.github/workflows/pre-commit.yaml
Outdated
@@ -13,4 +13,4 @@ jobs: | |||
uses: actions/checkout@v4 | |||
|
|||
- name: Run pre-commit checks | |||
uses: ccao-data/actions/pre-commit@main | |||
uses: ccao-data/actions/pre-commit@add-step-to-troubleshoot-intermittent-caching-errors |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will revert this after review
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remember to revert the temporary changes before merging, otherwise this is good to go!
.pre-commit-config.yaml
Outdated
#- repo: https://github.com/adrienverge/yamllint | ||
# rev: v1.32.0 | ||
# hooks: | ||
# - id: yamllint |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[Question, non-blocking] Out of curiosity, how does the branch name affect the yamllint
hook?
#30 (comment), @jeancochrane the length of the branch name when it is referenced in |
This PR adds a step in the pre-commit action that prints permissions of files in the R cache directory. Hopefully this helps us debug known cache issues.
Successful run here