[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

Lint

Lint Not Applicable

Unit

Tests Not Applicable

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 7900

View Options

arcanist/linter/PythonFormatLinter.php

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