You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
the bot could leave a comment of the Actions run when it starts
the bot could leave a comment noting whether the tests passed or failed
the workflow could be parallelized (again) - static analysis (shellcheck/spotbugs/spotless) can be done separately, with the unit testing and OCI packaging done separately from that with no dependency on static analysis, then integration testing and registry pushing can be done separately with only a dependency on the OCI packaging.
there could be a separate comment for retrying integration tests using the most recently-built images, ie /build_test first, then /retest to simply rerun the tests without a rebuild, if the reviewer suspects that the tests are simply flaky
is it possible for the /build_test to add a Status Check to the PR? Since moving these checks off from the pull_request_target and into a comment trigger, the checks do not seem to be linked anymore.
Anything other information?
No response
The text was updated successfully, but these errors were encountered:
I think it would also be nice for the automatic CI run on every PR update (every push to the PR branch) to still trigger the headless compile and unit tests (ie. mvn -Dheadless clean test), just skip the image build/push and integration testing until manually triggered.
Describe the feature
Since https://github.com/cryostatio/cryostat/pull/1599 , reviewers can comment
/build_test
on PRs to trigger a CI run for integration tests and preview images. For example,It would be nice if:
/build_test
first, then/retest
to simply rerun the tests without a rebuild, if the reviewer suspects that the tests are simply flaky/build_test
to add a Status Check to the PR? Since moving these checks off from thepull_request_target
and into a comment trigger, the checks do not seem to be linked anymore.Anything other information?
No response
The text was updated successfully, but these errors were encountered: