[LINTER] Escape the % char in the description of PythonFormatLinter
ClosedPublic
Actions

Authored by Fabien on Mar 31 2019, 08:13.

Details

Reviewers

deadalnix
jasonbcox

Group Reviewers

Restricted Project

Commits

rSTAGING9d53b39b2564: [LINTER] Escape the % char in the description of PythonFormatLinter
rABC9d53b39b2564: [LINTER] Escape the % char in the description of PythonFormatLinter

Summary

The % character in the getInfoDescription() method needs to be
escaped. Otherwise it throws an exception when running arc linters.

Test Plan

arc linters lint-python-format

Check the description string is displayed correcty.

Diff Detail

Repository

rABC Bitcoin ABC

Branch

python_format_lint_percent

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 5337
Build 8736: Bitcoin ABC Buildbot (legacy)
Build 8735: arc lint + arc unit

Event Timeline

Fabien created this revision.Mar 31 2019, 08:13

Herald added a reviewer: Restricted Project. · View Herald TranscriptMar 31 2019, 08:13

Herald added a reviewer: deadalnix. · View Herald Transcript

Herald added a subscriber: schancel. · View Herald Transcript

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing started.

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing passed.

Harbormaster completed remote builds in B5337: Diff 7882.Mar 31 2019, 08:28

jasonbcox accepted this revision.Apr 2 2019, 00:59

This revision is now accepted and ready to land.Apr 2 2019, 00:59

Closed by commit rABC9d53b39b2564: [LINTER] Escape the % char in the description of PythonFormatLinter (authored by Fabien). · Explain WhyApr 2 2019, 09:01

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rSTAGING9d53b39b2564: [LINTER] Escape the % char in the description of PythonFormatLinter.Sep 18 2019, 22:50

Revision Contents
Changeset List

Path

Size

arcanist/

linter/

PythonFormatLinter.php

2 lines

Diff 7882

View Options

arcanist/linter/PythonFormatLinter.php

[LINTER] Escape the % char in the description of PythonFormatLinterClosedPublicActions