Skip to content

Show error in Actions summary for safety issues in normal/scheduled runs#2138

Merged
andy-maier merged 1 commit into
masterfrom
andy/safety-warning
May 9, 2026
Merged

Show error in Actions summary for safety issues in normal/scheduled runs#2138
andy-maier merged 1 commit into
masterfrom
andy/safety-warning

Conversation

@andy-maier
Copy link
Copy Markdown
Member

@andy-maier andy-maier commented Apr 12, 2026

For details, see the commit message.

@andy-maier andy-maier self-assigned this Apr 12, 2026
@andy-maier andy-maier added this to the 1.26.0 milestone Apr 12, 2026
@andy-maier andy-maier changed the title Show error in Actions summary for safety issues in normal/schedules runs Show error in Actions summary for safety issues in normal/scheduled runs Apr 12, 2026
@coveralls
Copy link
Copy Markdown
Collaborator

coveralls commented Apr 12, 2026

Coverage Status

coverage: 77.19%. remained the same — andy/safety-warning into master

@andy-maier andy-maier force-pushed the andy/safety-warning branch 8 times, most recently from dfbffa4 to 197df15 Compare April 12, 2026 13:41
@andy-maier andy-maier force-pushed the andy/safety-warning branch 2 times, most recently from ad48e4c to fef2a4d Compare April 12, 2026 15:50
Details:

* Safety issues that are detected in normal and scheduled Actions runs
  now cause an error to be shown in the Actions summary. They still
  (intentionally) do not cause the Actions run to fail. Note that safety issues
  detected during an Actions release run, or during local use, do cause the
  make command and Actions run to fail.

* In addition, the safety command is now always run for both development
  and install before checking for failure.

Signed-off-by: Andreas Maier <maiera@de.ibm.com>
@andy-maier andy-maier dismissed vontedduchaithra’s stale review May 9, 2026 09:36

The merge-base changed after approval.

@andy-maier andy-maier force-pushed the andy/safety-warning branch from fef2a4d to 3eda470 Compare May 9, 2026 09:36
@andy-maier andy-maier merged commit 63d82e6 into master May 9, 2026
15 checks passed
@andy-maier andy-maier deleted the andy/safety-warning branch May 9, 2026 10:13
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