Page MenuHomePhabricator
Differential D15123

Fix linter tests
Needs ReviewPublic
Actions

Authored by vm on Jan 28 2016, 12:14 AM.
Tags
None
Referenced Files
F15267026: D15123.id36513.diff
Wed, Feb 26, 6:37 AM
Unknown Object (File)
Wed, Feb 19, 3:55 AM
Unknown Object (File)
Wed, Feb 19, 3:55 AM
Unknown Object (File)
Wed, Feb 19, 3:53 AM
Unknown Object (File)
Mon, Feb 17, 4:58 PM
Unknown Object (File)
Sun, Feb 2, 9:52 PM
Unknown Object (File)
Wed, Jan 29, 1:42 PM
Unknown Object (File)
Jan 15 2025, 7:37 PM
View All Files
Subscribers
Korvin

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Summary

ArcanistClosureLinterTestCase was not updated with D14010 which
changed the API for ArcanistLinterTestCase.
ArcanistCpplintLinterTestCase was failing with Cppcheck 1.72

Test Plan

Linter test cases pass

Diff Detail

Repository
rARC Arcanist
Branch
fix_lint_test
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 10353
Build 12624: arc lint + arc unit

Event Timeline

vm updated this revision to Diff 36513.Jan 28 2016, 12:14 AM
vm retitled this revision from to Fix linter tests.
vm updated this object.
vm edited the test plan for this revision. (Show Details)
vm added a reviewer: epriestley.
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptJan 28 2016, 12:14 AM
Herald added a subscriber: Korvin. · View Herald Transcript

Revision Contents
Changeset List

PathSize
src/
lint/
linter/
__tests__/
9 lines
cppcheck/
1 line

Diff 36513

View Options

src/lint/linter/__tests__/ArcanistClosureLinterTestCase.php

Loading...
View Options

src/lint/linter/__tests__/cppcheck/zblair.lint-test

Loading...
Phabricator · Built by Phacility