Differential D8749

utils/highlightjs: Do not report pygments exception to sentry
ClosedPublic
Actions

Authored by anlambert on Oct 19 2022, 4:42 PM.

Details

Reviewers

ardumont

Group Reviewers

Reviewers

Commits

rDWAPPSa2526ef5bb45: utils/highlightjs: Do not report pygments exception to sentry

Summary

This type of exception should not be reported to sentry as it is expected.

Diff Detail

Repository

rDWAPPS Web applications

Branch

master

Lint

No Linters Available

Unit

No Unit Test Coverage

Build Status

Buildable 32483
Build 50868: Phabricator diff pipeline on jenkins	Jenkins console · Jenkins
Build 50867: arc lint + arc unit

Event Timeline

anlambert created this revision.Oct 19 2022, 4:42 PM

Herald added a reviewer: Reviewers. · View Herald TranscriptOct 19 2022, 4:42 PM

Build is green

Patch application report for D8749 (id=31529)

Rebasing onto ef821584d5...

Current branch diff-target is up to date.

Changes applied before test

commit a2526ef5bb454b5facf8c6dc2b6a7a2490251ebd
Author: Antoine Lambert <anlambert@softwareheritage.org>
Date:   Wed Oct 19 16:42:24 2022 +0200

    utils/highlightjs: Do not report pygments exception to sentry
    
    This type of exception should not be reported to sentry as it is expected.

See https://jenkins.softwareheritage.org/job/DWAPPS/job/tests-on-diff/2082/ for more details.

Harbormaster completed remote builds in B32483: Diff 31529.Oct 19 2022, 4:51 PM

anlambert requested review of this revision.Oct 19 2022, 4:51 PM

ardumont accepted this revision.Oct 20 2022, 11:34 AM

This revision is now accepted and ready to land.Oct 20 2022, 11:34 AM

Closed by commit rDWAPPSa2526ef5bb45: utils/highlightjs: Do not report pygments exception to sentry (authored by anlambert). · Explain WhyOct 21 2022, 12:29 PM

This revision was automatically updated to reflect the committed changes.

anlambert added a commit: rDWAPPSa2526ef5bb45: utils/highlightjs: Do not report pygments exception to sentry.