Skip to content

Setup CodeQL Slack notifications#1354

Merged
sayboras merged 1 commit intocilium:mainfrom
kyle-c-simmons:pr/kyle-c-simmons/codeql-slack-notifications
May 28, 2025
Merged

Setup CodeQL Slack notifications#1354
sayboras merged 1 commit intocilium:mainfrom
kyle-c-simmons:pr/kyle-c-simmons/codeql-slack-notifications

Conversation

@kyle-c-simmons
Copy link
Contributor

Setup CodeQL Slack Notifications.

@kyle-c-simmons kyle-c-simmons requested a review from a team as a code owner May 21, 2025 12:55
@kyle-c-simmons kyle-c-simmons requested a review from jrajahalme May 21, 2025 12:55
@sayboras sayboras marked this pull request as draft May 21, 2025 12:58
@kyle-c-simmons kyle-c-simmons force-pushed the pr/kyle-c-simmons/codeql-slack-notifications branch 7 times, most recently from 438de3e to bd7f5f4 Compare May 21, 2025 15:25
@sayboras
Copy link
Member

you can mark it as draft PR as well, this will help to avoid any unnecessary notification while the changes are being developed.

@kyle-c-simmons kyle-c-simmons force-pushed the pr/kyle-c-simmons/codeql-slack-notifications branch 19 times, most recently from 189501d to b9e38b1 Compare May 27, 2025 09:35
@kyle-c-simmons kyle-c-simmons force-pushed the pr/kyle-c-simmons/codeql-slack-notifications branch 13 times, most recently from a12d660 to aae5704 Compare May 27, 2025 13:50
@kyle-c-simmons kyle-c-simmons changed the title Draft: Setup CodeQL Slack notifications Setup CodeQL Slack notifications May 27, 2025
@kyle-c-simmons kyle-c-simmons force-pushed the pr/kyle-c-simmons/codeql-slack-notifications branch from aae5704 to 361b43b Compare May 28, 2025 12:03
@kyle-c-simmons kyle-c-simmons marked this pull request as ready for review May 28, 2025 12:09
@kyle-c-simmons kyle-c-simmons force-pushed the pr/kyle-c-simmons/codeql-slack-notifications branch 2 times, most recently from d8e334a to 9bdce31 Compare May 28, 2025 12:20
Signed-off-by: kysimmon <kylesimmons96@protonmail.com>
@kyle-c-simmons kyle-c-simmons force-pushed the pr/kyle-c-simmons/codeql-slack-notifications branch from 9bdce31 to edaf9b7 Compare May 28, 2025 12:20
Copy link
Member

@sayboras sayboras left a comment

Choose a reason for hiding this comment

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

I reckon we need to merge this in and then verify EOD.

@kyle-c-simmons
Copy link
Contributor Author

@sayboras Are you able to merge this? I do not have permission to merge

@sayboras sayboras merged commit e978250 into cilium:main May 28, 2025
6 checks passed
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.

2 participants

Comments