Page MenuHomePhabricator
Differential D13931

ArcanistPyLintLinter fix to getMandatoryFlags msg-template
ClosedPublic
Actions

Authored by • e-m-albright on Aug 18 2015, 8:50 PM.
Tags
None
Referenced Files
F15461279: D13931.id.diff
Tue, Apr 1, 6:57 AM
F15396256: D13931.id33627.diff
Sun, Mar 16, 1:01 PM
Unknown Object (File)
Feb 11 2025, 6:04 PM
Unknown Object (File)
Feb 11 2025, 6:02 PM
Unknown Object (File)
Feb 11 2025, 1:21 PM
Unknown Object (File)
Feb 9 2025, 2:31 PM
Unknown Object (File)
Feb 8 2025, 10:08 PM
Unknown Object (File)
Feb 8 2025, 10:08 PM
View All Files
Subscribers
• e-m-albright
joshuaspence
Korvin

Details

Reviewers
joshuaspence
epriestley
Group Reviewers
Blessed Reviewers
Maniphest Tasks
T9214: ArcanistPyLintLinter.php has superfluous quotations around the "msg-template" options.
Commits
rARC05aaa1a5a3b8: ArcanistPyLintLinter fix to getMandatoryFlags msg-template
Required Signatures
L28 Phacility Individual Contributor License Agreement
Summary

Removed excess quotations on the --msg-template option as it was interfering with the string-int coercion

Test Plan

Unsure

Diff Detail

Repository
rARC Arcanist
Branch
master
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 7672
Build 8381: [Placeholder Plan] Wait for 30 Seconds
Build 8380: arc lint + arc unit

Event Timeline

• e-m-albright updated this revision to Diff 33627.Aug 18 2015, 8:50 PM
• e-m-albright retitled this revision from to ArcanistPyLintLinter fix to getMandatoryFlags msg-template.
• e-m-albright updated this object.
• e-m-albright edited the test plan for this revision. (Show Details)
• e-m-albright added a reviewer: epriestley.
• e-m-albright added subscribers: • e-m-albright, joshuaspence.
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptAug 18 2015, 8:50 PM
Herald added a required legal document: L28 Phacility Individual Contributor License Agreement. · View Herald Transcript
Herald added a subscriber: Korvin. · View Herald Transcript
• e-m-albright added a task: T9214: ArcanistPyLintLinter.php has superfluous quotations around the "msg-template" options..Aug 18 2015, 8:51 PM
• e-m-albright added a comment.Aug 18 2015, 8:55 PM

Apologies for the lack of unit testing and linting. I wouldn't think to submit this without them if they were anything more complicated than losing two quotation marks.

If I'm making any mistakes in the process here, I'm happy to learn/correct them!

Thanks again.

joshuaspence accepted this revision.Aug 19 2015, 12:17 PM
joshuaspence added a reviewer: joshuaspence.

I ran the unit tests locally and they still pass.

This revision is now accepted and ready to land.Aug 19 2015, 12:17 PM
epriestley edited edge metadata.Aug 19 2015, 12:45 PM

Thanks!

epriestley accepted this revision.Aug 19 2015, 12:45 PM
epriestley edited edge metadata.
Closed by commit rARC05aaa1a5a3b8: ArcanistPyLintLinter fix to getMandatoryFlags msg-template (authored by ealbright <ealbright@terrapower.com>, committed by epriestley). · Explain WhyAug 19 2015, 12:45 PM
This revision was automatically updated to reflect the committed changes.
epriestley mentioned this in T9214: ArcanistPyLintLinter.php has superfluous quotations around the "msg-template" options..Aug 19 2015, 1:53 PM

Revision Contents
Changeset List

PathSize
src/
lint/
linter/
2 lines

Diff 33627

View Options

src/lint/linter/ArcanistPyLintLinter.php

Loading...
Phabricator · Built by Phacility