-
Notifications
You must be signed in to change notification settings - Fork 112
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
Add: topic check for all deploys #8963
Conversation
d6e6e1d
to
b5d68cb
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@@ -100,4 +103,5 @@ runs: | |||
thread_ts: "${{ inputs.thread_ts }}" | |||
text: | | |||
❌ Build pipeline failed | |||
Is blocked by channel topic: ${{ inputs.blocked || "n/a" }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: can we display this only if actually blocked?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure : It you tell me how (don't want to spend an hour iterating again :))
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
😅.
Haven't tested but I've seen stuff like this:
${{ inputs.blocked && format('Is blocked by channel topic: {0}', inputs.blocked) }}
96a2577
to
dae343e
Compare
Description
Apply the channel topic check for all deploys.
Risk
Low (testing with front-edge)
Deploy Plan
Nothing