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

ci: Updated the PR checks to handle forks/dependabot. #12410

Conversation

rbarker-dev
Copy link
Contributor

Description:

This PR modifies .github/workflows/node-flow-pull-request-checks.yaml in order to handle PR checks from forks/dependabot.

  • Updated all the if statements in the node-flow-pull-request-checks.yaml file to provide a compatible check for pull requests, forks, and dependabot prs.

Related issue(s):

Fixes #11776

Notes for reviewer:

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

…equest-checks.yaml to handle PR checks from forks/dependabot.

Signed-off-by: Roger Barker <roger.barker@swirldslabs.com>
@rbarker-dev rbarker-dev requested a review from nathanklick March 27, 2024 19:18
@rbarker-dev rbarker-dev requested review from a team as code owners March 27, 2024 19:18
@rbarker-dev rbarker-dev linked an issue Mar 27, 2024 that may be closed by this pull request
@rbarker-dev rbarker-dev changed the title ci: Updated the if statements in ./.github/workflows/node-flow-pull-request-checks.yaml to handle PR checks from forks/dependabot. ci: Updated the PR checks to handle forks/dependabot. Mar 27, 2024
Copy link

Node: HAPI Test (Restart) Results

2 tests   2 ✔️  6m 33s ⏱️
2 suites  0 💤
2 files    0

Results for commit fb99776.

Copy link

Node: HAPI Test (Node Death Reconnect) Results

2 tests   2 ✔️  6m 50s ⏱️
2 suites  0 💤
2 files    0

Results for commit fb99776.

Copy link

Node: HAPI Test (Token) Results

202 tests   202 ✔️  18m 51s ⏱️
  16 suites      0 💤
  16 files        0

Results for commit fb99776.

Copy link

Node: HAPI Test (Crypto) Results

308 tests   308 ✔️  40m 3s ⏱️
  25 suites      0 💤
  25 files        0

Results for commit fb99776.

Copy link

Node: HAPI Test (Misc) Results

445 tests   434 ✔️  46m 35s ⏱️
  77 suites    10 💤
  77 files        1

For more details on these failures, see this check.

Results for commit fb99776.

Copy link

Node: HAPI Test (Time Consuming) Results

21 tests   21 ✔️  54m 22s ⏱️
  3 suites    0 💤
  3 files      0

Results for commit fb99776.

Copy link

Node: HAPI Test (Smart Contract) Results

504 tests   501 ✔️  59m 23s ⏱️
  55 suites      3 💤
  55 files        0

Results for commit fb99776.

Copy link

Node: Unit Test Results

    2 269 files  ±0      2 269 suites  ±0   2h 8m 4s ⏱️ - 48m 51s
112 199 tests ±0  112 103 ✔️ ±0  96 💤 ±0  0 ±0 
120 681 runs  ±0  120 584 ✔️ ±0  97 💤 ±0  0 ±0 

Results for commit fb99776. ± Comparison against base commit 5f4de86.

@rbarker-dev rbarker-dev merged commit 45d751e into develop Mar 27, 2024
39 of 42 checks passed
@rbarker-dev rbarker-dev deleted the 11776-adjust-the-dependabotfork-handling-in-the-pr-check-workflows branch March 27, 2024 22:42
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.

Adjust the dependabot/fork handling in the PR Check workflows
2 participants