Skip to content
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.

Sign up for GitHub

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

Jump to bottom

PR dashboard rarely commenting #10923

Closed
gsnedders opened this issue May 9, 2018 · 1 comment
Closed

PR dashboard rarely commenting #10923

gsnedders opened this issue May 9, 2018 · 1 comment
Labels
infra priority:roadmap

Comments

@gsnedders
Copy link
Member

The PR dashboard has stopped commenting on most PRs since 6 April. That said, nobody has complained about it in over a month so maybe it's not very valuable?

See https://github.com/w3c/web-platform-tests/pulls?page=1&q=is%3Apr+-commenter%3Aw3c-bots+created%3A%3E2017-04-05+-status%3Apending&utf8=✓ for everything where it has failed to comment since it started being run.

It seems like this might've been its first anniversary, so did something expire?
@gsnedders gsnedders added the infra label May 9, 2018
@gsnedders gsnedders mentioned this issue May 14, 2018
Closed
@foolip
Copy link
Member

foolip commented May 17, 2018

@lukebjerring @jugglinmike and I have discussed this. Conclusion is that we're OK with these comments not happening, because they were not terribly actionable, and we expect the responsibility of collecting/showing PR results to move to wpt.fyi in a matter of months.

So, we should do as little as possible, but probably should remove the bit of code that notifies the PR dashboard.

foolip added a commit that referenced this issue May 17, 2018
@foolip
Stop posting PR results to pulls.web-platform-tests.org
fb020ad 
Resolves #10923.
@foolip foolip mentioned this issue May 17, 2018
Merged
jugglinmike pushed a commit that referenced this issue May 18, 2018
@foolip @jugglinmike
Stop posting PR results to pulls.web-platform-tests.org (#11060)
4707a95 
Resolves #10923.
moz-v2v-gh pushed a commit to mozilla/gecko-dev that referenced this issue May 22, 2018
@foolip @moz-wptsync-bot
Bug 1462524 [wpt PR 11060] - Stop posting PR results to pulls.web-pla…
637b01a 
…tform-tests.org, a=testonly

Automatic update from web-platform-testsStop posting PR results to pulls.web-platform-tests.org (#11060)

Resolves web-platform-tests/wpt#10923.
--

wpt-commits: 4707a955260a76af42a0af32b9547c277911a9d9
wpt-pr: 11060
jugglinmike added a commit to bocoup/infrastructure-web-platform that referenced this issue Jan 18, 2019
@jugglinmike
Decommission pr-dashboard project
511e961 
The configuration which integrated this project with the
web-platform-tests was disabled on 2018-05-18 [1]. The application's
content is now only useful for historical reference, and no dynamism or
computation is necessary to fulfill that purpose. A snapshot of the
application's current content has been created and made available in the
project's source code repository. Bocoup will maintain that as a static
collection of HTML files published to the web.

Remove the infrastructure originally provisioned to power the dynamic
web application.

[1] web-platform-tests/wpt#10923
@jugglinmike jugglinmike mentioned this issue Jan 18, 2019
Merged
leobalter pushed a commit to bocoup/infrastructure-web-platform that referenced this issue Jul 29, 2019
@jugglinmike @leobalter
Decommission pr-dashboard project (#11)
e1cc139 
The configuration which integrated this project with the
web-platform-tests was disabled on 2018-05-18 [1]. The application's
content is now only useful for historical reference, and no dynamism or
computation is necessary to fulfill that purpose. A snapshot of the
application's current content has been created and made available in the
project's source code repository. Bocoup will maintain that as a static
collection of HTML files published to the web.

Remove the infrastructure originally provisioned to power the dynamic
web application.

[1] web-platform-tests/wpt#10923
gecko-dev-updater pushed a commit to marco-c/gecko-dev-comments-removed that referenced this issue Oct 3, 2019
@marco-c
Bug 1462524 [wpt PR 11060] - Stop posting PR results to pulls.web-pla…
5e3cc19 
…tform-tests.org, a=testonly

Automatic update from web-platform-testsStop posting PR results to pulls.web-platform-tests.org (#11060)

Resolves web-platform-tests/wpt#10923.
--

wpt-commits: 4707a955260a76af42a0af32b9547c277911a9d9
wpt-pr: 11060

UltraBlame original commit: 48b2204ff7fbd56f7bc9f071558ef445c3247787
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified that referenced this issue Oct 3, 2019
@marco-c
Bug 1462524 [wpt PR 11060] - Stop posting PR results to pulls.web-pla…
c5f0a91 
…tform-tests.org, a=testonly

Automatic update from web-platform-testsStop posting PR results to pulls.web-platform-tests.org (#11060)

Resolves web-platform-tests/wpt#10923.
--

wpt-commits: 4707a955260a76af42a0af32b9547c277911a9d9
wpt-pr: 11060

UltraBlame original commit: 48b2204ff7fbd56f7bc9f071558ef445c3247787
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified-and-comments-removed that referenced this issue Oct 3, 2019
@marco-c
Bug 1462524 [wpt PR 11060] - Stop posting PR results to pulls.web-pla…
3d7238c 
…tform-tests.org, a=testonly

Automatic update from web-platform-testsStop posting PR results to pulls.web-platform-tests.org (#11060)

Resolves web-platform-tests/wpt#10923.
--

wpt-commits: 4707a955260a76af42a0af32b9547c277911a9d9
wpt-pr: 11060

UltraBlame original commit: 48b2204ff7fbd56f7bc9f071558ef445c3247787
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
infra priority:roadmap
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@gsnedders @foolip