P5.js: Errors from fork emailed to users that merge pull requests

Created on 17 Feb 2020  路  3Comments  路  Source: processing/p5.js

The Twitter Together Github action is failing on @Mattlk13 's fork of p5.js and alerting anyone that makes a merge in the upstream (this repo) via email. The error is:

##[error]HttpError: Resource not accessible by integration

Because the fork is trying to run our twitter github actions but of course it cannot and should not. @gr2m has pointed out that it might be due to some automation on that user's fork.

devops bug

Most helpful comment

Oh I think I know what's going on. I check if the pull request head is a fork in which case I do not try to create a check run. In this particular pull request, the pull request head is not the fork, the base is the fork 馃 I might just add a check for that, too, to resolve the problem. I made an issue here: https://github.com/gr2m/twitter-together/issues/104

All 3 comments

Oh I think I know what's going on. I check if the pull request head is a fork in which case I do not try to create a check run. In this particular pull request, the pull request head is not the fork, the base is the fork 馃 I might just add a check for that, too, to resolve the problem. I made an issue here: https://github.com/gr2m/twitter-together/issues/104

This seems to be fixed over here. Closing for now.

yay :)

Was this page helpful?
0 / 5 - 0 ratings

Related issues

kappaxbeta picture kappaxbeta  路  3Comments

aman-tiwari picture aman-tiwari  路  3Comments

Vbbab picture Vbbab  路  3Comments

oranyelik picture oranyelik  路  3Comments

aferriss picture aferriss  路  3Comments