Skip to content

chore: Adjust conditions for running CI jobs - #261

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

scouten-adobe merged 4 commits into
mainfrom
adjust-ci-run-conditions

Conversation

@scouten-adobe

Copy link
Copy Markdown
Collaborator

No description provided.

Comment thread .github/workflows/ci.yml
- 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
Copy Markdown
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

codecov-commenter commented Oct 16, 2024 •

Copy link
Copy Markdown

⚠️ 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.

Comment thread .github/workflows/ci.yml
- opened
- reopened
- synchronize
- labeled

Copy link
Copy Markdown
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.

Comment thread .github/workflows/ci.yml
- cron: "30 18 * * 1,4,6" # 1830 UTC every Monday, Thursday, Saturday

jobs:
debug:

Copy link
Copy Markdown
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.

Comment thread .github/workflows/ci.yml
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
Copy Markdown
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.

@mauricefisher64 mauricefisher64 left a comment

Copy link
Copy Markdown
Contributor

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

3 participants