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

Don't run scheduled jobs on forks #709

Merged
merged 1 commit into from
Aug 9, 2023

Conversation

rgieseke
Copy link
Contributor

@rgieseke rgieseke commented Jul 1, 2023

Scheduled jobs should only run in the main repo.
Solution from https://github.com/orgs/community/discussions/26684

Let me know if you want me to experiment with the date for testing!

This closes #708

@codecov-commenter
Copy link

Codecov Report

Merging #709 (9f4e7bc) into master (8087ca7) will not change coverage.
The diff coverage is n/a.

❗ Current head 9f4e7bc differs from pull request most recent head 0fdb129. Consider uploading reports for the commit 0fdb129 to get more accurate results

@@           Coverage Diff           @@
##           master     #709   +/-   ##
=======================================
  Coverage   79.72%   79.72%           
=======================================
  Files          61       61           
  Lines        5978     5978           
=======================================
  Hits         4766     4766           
  Misses       1212     1212           

@kdorheim kdorheim requested a review from bpbond July 19, 2023 01:50
@kdorheim kdorheim merged commit 9027f1f into JGCRI:master Aug 9, 2023
@kdorheim
Copy link
Contributor

kdorheim commented Aug 9, 2023

@rgieseke thanks!

@rgieseke
Copy link
Contributor Author

rgieseke commented Aug 9, 2023

Thanks, i've synced my fork and will check next month!

@kdorheim
Copy link
Contributor

kdorheim commented Aug 9, 2023

Awesome! Keep us posted!

@rgieseke
Copy link
Contributor Author

rgieseke commented Sep 1, 2023

Scheduled jobs in my fork have been skipped! ✔️

image

@kdorheim
Copy link
Contributor

kdorheim commented Sep 1, 2023

Awesome!

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.

Scheduled Actions shouldn't run on forks
4 participants