-
Notifications
You must be signed in to change notification settings - Fork 18
Why doesn’t Peril seem to be actually adding labels? #22
Comments
Hmmm... It ran on Heroku, but nothing happened. I have to run now, but will look into adding logging later today. |
Checklist:
Other thoughts:
|
Is the app installed for all repositories or gatsby/selected repositories? |
I've started setting up peril on my own heroku account, so should be able to play around more with it tomorrow. |
Refs: #22 I had some issues with setting up peril for my account. Didn't have time to debug the setup, so at least I've added few debug logs + label without colon just to test. @jlengstorf @SD10
I updated Peril, and I started getting errors. Still can't get console logs out of the labeler task. Posted an issue at danger/peril#351 I'm at a bit of a loss at this point, so will wait to hear back on this issue. A few other things got weird as well (triple-firing, for example). Not sure if that's related to the work we've been doing here or something else, but need to figure out what's happening. |
I just rolled back in hopes of restoring the working PR integration. |
So this works now, but there seems to be a problem with running multiple tasks, and I'm still hitting errors in the latest release of Peril. Once @orta is back around I'll see if I can help track down how/why that's happening. For now, closing this. |
What about now? cc @piotrkwiecinski @SD10
The text was updated successfully, but these errors were encountered: