HomePhorge

Improved warning message when accepting diff with skipped lint or unit

Description

Improved warning message when accepting diff with skipped lint or unit

Summary:
Some people find the current message stating "This diff has Lint/Unit Test
Problems" confusing if the unit tests or lint was skipped. This revision
clarifies those messages.

Test Plan:
Started to accept a revision with skipped lint and unit tests, and saw the new
message.

Reviewers: epriestley, btrahan, jungejason

Reviewed By: epriestley

CC: aran, epriestley

Differential Revision: https://secure.phabricator.com/D1738

Details

Provenance
Nick HarperAuthored on Feb 27 2012, 4:09 PM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rP4c8f405fccc2: Use wide links only for line and blame-prev in Diffusion
Branches
Unknown
Tags
Unknown

Event Timeline