-
Notifications
You must be signed in to change notification settings - Fork 7.5k
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
chore(lock-threads): run only daily at 1:00 am, and skip in forks #7832
Conversation
Codecov Report
@@ Coverage Diff @@
## main #7832 +/- ##
=======================================
Coverage 80.93% 80.93%
=======================================
Files 116 116
Lines 7463 7463
Branches 1813 1813
=======================================
Hits 6040 6040
Misses 1423 1423 Continue to review full report at Codecov.
|
Thanks, this doesn't need to run often now the backlog is cleared. Could have |
I wish actions didn't run in forked repos by default. Can definitely be annoying. |
@@ -13,6 +13,7 @@ concurrency: | |||
|
|||
jobs: | |||
action: | |||
if: ${{ github.repository_owner == 'videojs' }} |
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.
This will cause the jobs to be skipped on forks. They'll still show in the actions list.
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.
@mister-ben thank you, that's even better !
…deojs#7832) * chore(lock-action): performed daily at 1:00 am * Skip for forks Co-authored-by: mister-ben <git@misterben.me>
Description
The github action introduced by #7777 in charge of locking old issues runs every hour and is followed by an e-mail.
Specific Changes proposed
The Lock Threads does not seem to allow restricting its execution to a named repo. The proposal would be to execute the action only once a day to avoid spamming those who have a fork of the project.
Requirements Checklist