HomePhabricator

(stable) When a linter raises a message at a nonexistent line, don't fatal…

Description

(stable) When a linter raises a message at a nonexistent line, don't fatal during rendering

Summary:
See PHI1782. If a linter raises a message at a line which does not exist in the file, render a confused warning rather than fataling.

This is a long-existing issue which was exacerbated by D21044.

Test Plan: Modified a linter to raise issues at line 99999. Before change: fatal in console rendering. After change: reasonable rendering.

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

Details

Provenance
epriestleyAuthored on Jun 12 2020, 7:05 PM
epriestleyPushed on Jun 12 2020, 7:31 PM
Differential Revision
D21357: When a linter raises a message at a nonexistent line, don't fatal during rendering
Parents
rARCb689a8c78b6c: (stable) Remove "--encoding" flag from "arc diff"
Branches
Unknown
Tags
Unknown
Build Status
Buildable 24658
Build 33999: Run Core Tests