This is an archive of the discontinued Mercurial Phabricator instance.

ci: report status to phabricator on failures
Needs RevisionPublic

Authored by danchr on Jan 12 2021, 7:08 AM.

Details

Reviewers
marmoute
Group Reviewers
hg-reviewers

Diff Detail

Repository
rHG Mercurial
Branch
default
Lint
No Linters Available
Unit
No Unit Test Coverage

Event Timeline

danchr created this revision.Jan 12 2021, 7:08 AM
baymax updated this revision to Diff 24719.Jan 12 2021, 8:11 AM

❎ refresh by Heptapod after a failed CI run (🐙 ❤️) https://foss.heptapod.net/octobus/mercurial-devel/-/pipelines/15994

baymax updated this revision to Diff 24872.Jan 14 2021, 9:18 PM

❎ refresh by Heptapod after a failed CI run (🐙 ❤️) https://foss.heptapod.net/octobus/mercurial-devel/-/pipelines/16444

marmoute requested changes to this revision.Jan 15 2021, 11:18 PM
marmoute added a subscriber: marmoute.

phabricator is inconstistent when it comes to updating diff (and printing associated message). So I fear we would end up in situation where a comment would be made to make a Diff as part of a failing run, without it being replaced by a positive message once test pass.

So -1 on this until it is fixed or proved to not happen.

This revision now requires changes to proceed.Jan 15 2021, 11:18 PM