-
Notifications
You must be signed in to change notification settings - Fork 17
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 suggestion to use cache if no depot is detected #41
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #41 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 1 1
Lines 2 2
=========================================
Hits 2 2 ☔ View full report in Codecov by Sentry. |
7bf2716
to
0153260
Compare
0153260
to
1f8cf1b
Compare
I don't think I know enough to review this PR. @DilumAluthge Can we disable the review requirements to make maintenance here easier? |
I think it's nice to require PR reviews, but I don't feel too strongly either way. @SaschaMann Thoughts on removing the required reviews? |
The dependabot ones are the annoying ones... |
I think we can configure Dependbot to auto-approve its own PRs. |
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.
LGTM
This comment was marked as resolved.
This comment was marked as resolved.
Can we set up dependabot to approve its own PRs? |
Closes #40
Adds a suggestion to use
julia-actions/cache
if no depot is detected.Can be disabled with
ignore-no-cache: true
If this is approved, I'll do the same for
julia-runtest
(whichever comes first in a run will show the note then create the depot, so we won't end up with double notifications)