-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
馃彈 馃毊 Remove test-status
bot reporting
#35249
馃彈 馃毊 Remove test-status
bot reporting
#35249
Conversation
Hey @rsimha! These files were changed:
Hey @estherkim! These files were changed:
|
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 after removing the debugging leftovers. Goodnight, sweet bot!
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.
Thanks for the cleanup. LGTM after one more deletion.
And appreciate all your past work on this, @danielrozenberg!
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.
Did you want to delete https://github.com/ampproject/amp-github-apps/tree/main/test-status too? 馃槵
And we can turn down the app engine
@estherkim we're planning on waiting a couple days before turning down the bot incase someone complains (can't really imagine that happening). I'll put up a PR deleting the code in |
661fb3b
to
bf089db
Compare
Since moving to CircleCI, the
test-status
bot has become more of a maintenance burden than a help due to the CircleCI GitHub integration providing the same information.#35247