Skip to content
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

Merged
merged 2 commits into from
Jul 11, 2022

Conversation

amtins
Copy link
Contributor

@amtins amtins commented Jul 7, 2022

Description

The github action introduced by #7777 in charge of locking old issues runs every hour and is followed by an e-mail.

github-lock-action

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

  • Feature implemented / Bug fixed
  • If necessary, more likely in a feature request than a bug fix
    • Change has been verified in an actual browser (Chrome, Firefox, IE)
    • Unit Tests updated or fixed
    • Docs/guides updated
    • Example created (starter template on JSBin)
  • Reviewed by Two Core Contributors

@codecov
Copy link

codecov bot commented Jul 7, 2022

Codecov Report

Merging #7832 (b8e7e2b) into main (1445f87) will not change coverage.
The diff coverage is n/a.

@@           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.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1445f87...b8e7e2b. Read the comment docs.

@mister-ben
Copy link
Contributor

Thanks, this doesn't need to run often now the backlog is cleared. Could have if: ${{ github.repository_owner == 'videojs' }} under action too.

@gkatsev
Copy link
Member

gkatsev commented Jul 7, 2022

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' }}
Copy link
Contributor

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.

Copy link
Contributor Author

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 !

@mister-ben mister-ben changed the title chore(lock-threads): performed daily at 1:00 am chore(lock-threads): run only daily at 1:00 am, and skip in forks Jul 11, 2022
@mister-ben mister-ben merged commit 1be46d4 into videojs:main Jul 11, 2022
@amtins amtins deleted the chore/lock-action branch July 11, 2022 12:58
misteroneill pushed a commit that referenced this pull request Nov 22, 2022
)

* chore(lock-action): performed daily at 1:00 am

* Skip for forks

Co-authored-by: mister-ben <git@misterben.me>
edirub pushed a commit to edirub/video.js that referenced this pull request Jun 8, 2023
…deojs#7832)

* chore(lock-action): performed daily at 1:00 am

* Skip for forks

Co-authored-by: mister-ben <git@misterben.me>
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.

4 participants