Remove the Pygments error marker for syntax-highlighted diffs.

Review Request #13290 — Created Sept. 26, 2023 and submitted

Information

Review Board
release-6.x

Reviewers

Pygments doesn't always recognize all syntax in a file, and while this
is often harmless, it comes with a lot of visual noise in the form of a
red border around the unexpected parts.

The indicator was more useful once upon a time, but with modern
practices around lint checking and CI builds, these markers are more
annoying than helpful.

This change removes visual indicators on unknown syntax, allowing
reviewers to concentrate more on the code without the distractions.

Posted a change with a LessCSS file that triggered Pygments error
markers. Saw that the change lacked the borders.

Summary ID
Remove the Pygments error marker for syntax-highlighted diffs.
Pygments doesn't always recognize all syntax in a file, and while this is often harmless, it comes with a lot of visual noise in the form of a red border around the unexpected parts. The indicator was more useful once upon a time, but with modern practices around lint checking and CI builds, these markers are more annoying than helpful. This change removes visual indicators on unknown syntax, allowing reviewers to concentrate more on the code without the distractions.
224693ddea5a2f7f08940133eb905ac91992d0f3
maubin
  1. Ship It!
  2. 
      
david
  1. Ship It!
  2. 
      
chipx86
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-6.x (dad8ab5)
Loading...