search

web-platform-tests / rfcs

web-platform-tests RFCs
75 stars 63 forks source link

RFC 35: Add third-party GitHub App: Pull Panda #35

Closed zcorpan closed 4 years ago

zcorpan commented 4 years ago

View the rendered version.

jgraham commented 4 years ago

Enabling a GH-provided app seems fine to me. Argubaly it doesn't even need an RFC, just agreement from the web-platform-tests/wpt admins.

zcorpan commented 4 years ago

I've clarified that we intend to enable this for the whole org (rather than only the wpt repo).

foolip commented 4 years ago

@zcorpan given that it's read-only, can you go ahead and enable it so we can see it live? This RFC can remain open for a week and we should turn it off if anyone raises concerns within that week.

zcorpan commented 4 years ago

Done. The app can be configured and uninstalled here:

https://github.com/organizations/web-platform-tests/settings/installations/4993614

Go to https://pullpanda.com/ and log in with your GitHub account. Choose "web-platform-tests" and continue to "Analytics". Click "Analytics" at the top again to view different graphs.

It probably takes a while for it to have all the data for some of the graphs.

zcorpan commented 4 years ago

I noticed now that the history is 24 weeks (almost 6 months), rather than 3 months.

zcorpan commented 4 years ago

Hmm. I wonder if http://github.com/web-platform-tests/wpt-actions-test/pull/17 broke Pull Panda. The "Longest time to merge" list includes only that PR, and some other graphs are missing data.

Edit: filed https://github.com/pullreminders/backlog/issues/208

zcorpan commented 4 years ago

I've changed the selected repos to wpt, wpt-pr-bot, wpt.fyi and rfcs, but I think the problem persists, maybe due to https://github.com/web-platform-tests/wpt/pull/20036 which also has an empty commit.

foolip commented 4 years ago

Funny how we seem to be able to break other tools with all the weird things we get up to in this repo :)

foolip commented 4 years ago

Enough time has passed, I'll merge this now.

zcorpan commented 4 years ago

I've filed another issue for Pull Panda about the lack of ability to filter away bot-created PRs in the analytics. https://github.com/pullreminders/backlog/issues/210