Raghu Simha
Raghu Simha
Hmm. After doing some more searching, I see that similar problems were encountered in #203. I'll leave this open until @siddharthkp gets a chance to take a look.
@siddharthkp I believe they registered as GitHub apps, so the permission is granted through the API and not through a token. See https://developer.github.com/apps/building-github-apps/
A summary of how to do this is in the [LGTM docs](https://lgtm.com/help/lgtm/using-lgtm-analysis-continuous-integration): > Pull request integration requires access to repository webhooks and services, and access to commit statuses.
@siddharthkp Any word on this? We're still looking for a way to enable native Github integration for PRs originating from forks.
@4kochi, thanks. I'll ping this bug if we see this again.
@4kochi Here's an update. I encountered several more instances of unexpected error reports from `karma-mocha-reporter`. In some cases, it reported several alleged failures even though only one test failed. It's...
This was originally tracked by #48, and partially fixed by #49.
@sebastianbenz Looks like this needs an accompanying change to the CI config, so will leave this to the maintainers of this repo.
I think this can be covered by E2E tests now. @prateekbh Can you comment?
Reopening because this test is repeatedly failing again. E.g. https://app.circleci.com/pipelines/github/ampproject/amphtml/1549/workflows/cdbc541d-1fdb-467a-9a28-6612a08f542f/jobs/14788/parallel-runs/0/steps/0-108 /cc @ampproject/wg-monetization