Skip to content

tools: actually _notifying_ first pass reviewers#17964

Merged
mattklein123 merged 1 commit intoenvoyproxy:mainfrom
alyssawilk:notifier
Sep 3, 2021
Merged

tools: actually _notifying_ first pass reviewers#17964
mattklein123 merged 1 commit intoenvoyproxy:mainfrom
alyssawilk:notifier

Conversation

@alyssawilk
Copy link
Copy Markdown
Contributor

No description provided.

Signed-off-by: Alyssa Wilk <alyssar@chromium.org>
@alyssawilk
Copy link
Copy Markdown
Contributor Author

this time actually tested with slackbot token and confirmed with @RyanTheOptimist
sigh.

Copy link
Copy Markdown
Contributor

@RyanTheOptimist RyanTheOptimist left a comment

Choose a reason for hiding this comment

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

Thanks for doing this!

@mattklein123 mattklein123 merged commit 3a507f2 into envoyproxy:main Sep 3, 2021
tyxia pushed a commit to tyxia/envoy that referenced this pull request Sep 21, 2021
Signed-off-by: Alyssa Wilk <alyssar@chromium.org>

Signed-off-by: alyssawilk <alyssar@chromium.org>
@alyssawilk alyssawilk deleted the notifier branch February 28, 2022 21:25
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.

4 participants