• 
      

    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:
    Completed
    Change Summary:
    Pushed to release-6.x (dad8ab5)