@recaptime-dev's working patches + fork for Phorge, a community fork of Phabricator. (Upstream dev and stable branches are at upstream/main and upstream/stable respectively.)
hq.recaptime.dev/wiki/Phorge
phorge
phabricator
Mark finished postponed linters as okay
Test Plan: Called it on a diff with postponed linters and no messages.
Reviewers: mgummelt, epriestley
Reviewed By: epriestley
CC: aran, Korvin
Differential Revision: https://secure.phabricator.com/D4223
+1
-2
+1
-2
src/applications/conduit/method/differential/ConduitAPI_differential_setdiffproperty_Method.php
+1
-2
src/applications/conduit/method/differential/ConduitAPI_differential_setdiffproperty_Method.php
···
77
77
$diff->setLintStatus(DifferentialLintStatus::LINT_WARN);
78
78
} else if (!empty($postponed_linters)) {
79
79
$diff->setLintStatus(DifferentialLintStatus::LINT_POSTPONED);
80
-
} else if ($results &&
81
-
$diff->getLintStatus() === DifferentialLintStatus::LINT_NONE) {
80
+
} else if ($diff->getLintStatus() != DifferentialLintStatus::LINT_SKIP) {
82
81
$diff->setLintStatus(DifferentialLintStatus::LINT_OKAY);
83
82
}
84
83
$diff->save();