Let default parameters linter rule support `...` in variadic function
Needs ReviewPublic
Actions

Authored by wjiang on May 4 2017, 2:12 AM.

Tags

None

Referenced Files

	F19012774: D17825.id42871.diff
	Sat, Nov 22, 2:05 PM

	F19012480: D17825.id42871.diff
	Sat, Nov 22, 1:12 PM

	F19002492: D17825.id42870.diff
	Fri, Nov 21, 6:16 AM

	F18851292: D17825.diff
	Thu, Oct 30, 11:35 PM

	F18814597: D17825.diff
	Oct 20 2025, 11:34 PM

	F18778982: D17825.id42870.diff
	Oct 11 2025, 11:31 AM

	F18778979: D17825.id42871.diff
	Oct 11 2025, 11:31 AM

	F18777558: D17825.diff
	Oct 11 2025, 3:37 AM

View All Files

Subscribers

epriestley

joshuaspence

Details

Reviewers

None

Group Reviewers

Blessed Reviewers

Maniphest Tasks

T12667: Let default parameters linter rule support `...` in variadic function

Summary

Ref T12667. As titled.

Test Plan

Run added unit test.

Diff Detail

Repository

rARC Arcanist

Branch

arcpatch-D17825

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 16797
Build 22417: arc lint + arc unit

Event Timeline

wjiang created this revision.May 4 2017, 2:12 AM

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptMay 4 2017, 2:12 AM

Herald added a subscriber: epriestley. · View Herald Transcript

wjiang added a subscriber: joshuaspence.May 4 2017, 2:13 AM

Minor

Revision Contents
Changeset List

Path

Size

Coverage (All)

Coverage (Touched)

src/

lint/

linter/

xhpast/

rules/

ArcanistDefaultParametersXHPASTLinterRule.php

4 lines

100%

__tests__/

default-parameters/

default-parameters.lint-test

2 lines

Diff 42871

View Options

src/lint/linter/xhpast/rules/ArcanistDefaultParametersXHPASTLinterRule.php

View Options

src/lint/linter/xhpast/rules/tests/default-parameters/default-parameters.lint-test

Let default parameters linter rule support `...` in variadic functionNeeds ReviewPublicActions