-
Notifications
You must be signed in to change notification settings - Fork 11
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
testing new all-contributors app #163
Comments
Hello there from Cloudflare Workers |
@all-contributors please add @gr2m-test for infrastructure, tests and code |
I've put up a pull request to add @gr2m-test! 🎉 |
@all-contributors please add @gr2m-test for infrastructure, tests and code |
I've updated the pull request to add @gr2m-test! 🎉 |
@all-contributors please add @gr2m-test for code |
I've updated the pull request to add @gr2m-test! 🎉 |
@all-contributors please add @gr2m-test for tests |
I've updated the pull request to add @gr2m-test! 🎉 |
@all-contributors please add @gr2m-test for infrastructure |
I've updated the pull request to add @gr2m-test! 🎉 |
@all-contributors please add @gr2m-test for infrastructure |
I've put up a pull request to add @gr2m-test! 🎉 |
@all-contributors please add @gr2m-test for tests |
I've put up a pull request to add @gr2m-test! 🎉 |
@all-contributors please add @gr2m for tests |
I've put up a pull request to add @gr2m! 🎉 |
@all-contributors please add @Berkmann18 for mentoring |
I've put up a pull request to add @Berkmann18! 🎉 |
@all-contributors please add @Berkmann18 for code |
We had trouble processing your request. Please try again later. |
the error was
|
It should have been reported to sentry, the sentry event ID is |
It may be taking time in showing up. |
I think it's the same error that you see in the tests here? https://github.com/all-contributors/app/pull/343/checks?check_run_id=1723922388#step:5:13 |
That's true and it indeed seems that it only fails in the PR. |
@all-contributors please add @Berkmann18 for code |
I've updated the pull request to add @Berkmann18! 🎉 |
Note (to self): check if the PR title is consistent with the commits when (the convention isn't Angular) if so the PR would be good to go. |
@all-contributors please add @gr2m for infra |
I've updated the pull request to add @gr2m! 🎉 |
It seems like we need to have tests that read a mocked/dummy |
looks like #167 was created by the @allcontributors app, not by @all-contributors-preview. Sorry, my fault. I installed the @allcontributors app all of my @gr2m repositories and forgot that I tested the preview here in this repo. How about we create a new organization @all-contributors-sandbox for dedicated testing? |
I could not determine your intention. Basic usage: @all-contributors please add @Someone for code, doc and infra For other usages see the documentation |
@all-contributors-preview[bot] I've put up a pull request to add @Someone! 🎉 |
oh oh :D |
No description provided.
The text was updated successfully, but these errors were encountered: