Skip to content

CI test workflow - #192

Merged
JPadovano1483 merged 3 commits into
skupperproject:mainfrom
JPadovano1483:james-test-reporter
Aug 10, 2026
Merged

CI test workflow#192
JPadovano1483 merged 3 commits into
skupperproject:mainfrom
JPadovano1483:james-test-reporter

Conversation

@JPadovano1483

@JPadovano1483 JPadovano1483 commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

fixes #191

@@ -0,0 +1,106 @@
name: Test Suite

on:

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Should this new action be constrained to run on the upstream repo only?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

It's not a requirement like the image workflow since it doesn't rely on repo secrets, but it probably doesn't add much value to have it run on our forks and may just add noise when rebasing. I can add that constraint.

@fgiorgetti fgiorgetti left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@JPadovano1483
JPadovano1483 requested a review from ted-ross August 7, 2026 17:33
@ted-ross

ted-ross commented Aug 7, 2026

Copy link
Copy Markdown
Member

Why is the CI job for this PR not running?

@JPadovano1483

JPadovano1483 commented Aug 7, 2026

Copy link
Copy Markdown
Contributor Author

Why is the CI job for this PR not running?

An excellent question. Github actions was having some issues yesterday and the workflow was failing to startup. Looking in the "actions" tab, the workflow says it's queued and fails to cancel. I'll try closing and reopening...

@JPadovano1483
JPadovano1483 merged commit 06453f3 into skupperproject:main Aug 10, 2026
2 of 6 checks passed
@JPadovano1483
JPadovano1483 deleted the james-test-reporter branch August 10, 2026 16:53
This was referenced Aug 10, 2026
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.

run unit and integration tests on PR

3 participants