Skip to content
This repository was archived by the owner on Dec 11, 2024. It is now read-only.

chore: Adjust conditions for running CI jobs#261

Merged
scouten-adobe merged 4 commits intomainfrom
adjust-ci-run-conditions
Oct 16, 2024
Merged

chore: Adjust conditions for running CI jobs#261
scouten-adobe merged 4 commits intomainfrom
adjust-ci-run-conditions

Conversation

@scouten-adobe
Copy link
Collaborator

No description provided.

- nightly
schedule:
- cron: "0 18 * * 1,4,6" # 1800 UTC every Monday, Thursday, Saturday
- cron: "30 18 * * 1,4,6" # 1830 UTC every Monday, Thursday, Saturday
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not totally related to original purpose, but moving this 30 minutes later to ensure that we pick up c2pa-rs nightly which also runs at 1800 UTC.

@codecov-commenter
Copy link

codecov-commenter commented Oct 16, 2024

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 41.79%. Comparing base (17863bd) to head (2d77395).
Report is 22 commits behind head on main.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

❗ There is a different number of reports uploaded between BASE (17863bd) and HEAD (2d77395). Click for more details.

HEAD has 9 uploads less than BASE
Flag BASE (17863bd) HEAD (2d77395)
12 3
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #261      +/-   ##
==========================================
- Coverage   48.68%   41.79%   -6.89%     
==========================================
  Files           4        4              
  Lines         684      756      +72     
==========================================
- Hits          333      316      -17     
- Misses        351      440      +89     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

- opened
- reopened
- synchronize
- labeled
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will cause PR tests to run automatically once somebody adds the "safe to test" label.

- cron: "30 18 * * 1,4,6" # 1830 UTC every Monday, Thursday, Saturday

jobs:
debug:
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll remove this once we have verified intended behavior for external PRs.

if: contains(github.event.pull_request.labels.*.name, 'safe to test')
if: |
contains(github.event.pull_request.labels.*.name, 'safe to test') ||
github.event.pull_request.author_association == 'COLLABORATOR' ||
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

COLLABORATOR means anyone who is explicitly allow-listed in the repo Settings page.

MEMBER means anyone who is allow-listed in the organization's Settings page.

Copy link
Contributor

@mauricefisher64 mauricefisher64 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@scouten-adobe scouten-adobe merged commit 50e6c83 into main Oct 16, 2024
@scouten-adobe scouten-adobe deleted the adjust-ci-run-conditions branch October 16, 2024 19:45
scouten-adobe added a commit to contentauth/c2pa-rs that referenced this pull request Dec 10, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants