Page MenuHomePhabricator
Differential D12921

Improve the robustness of the "alias function" linter rule
ClosedPublic
Actions

Authored by joshuaspence on May 19 2015, 8:12 AM.
Tags
None
Referenced Files
F15454501: D12921.diff
Sat, Mar 29, 6:50 PM
F15421434: D12921.id31084.diff
Fri, Mar 21, 11:45 PM
F15388044: D12921.id31161.diff
Sat, Mar 15, 3:14 AM
Unknown Object (File)
Feb 25 2025, 4:58 AM
Unknown Object (File)
Feb 20 2025, 4:28 AM
Unknown Object (File)
Feb 17 2025, 10:36 AM
Unknown Object (File)
Feb 11 2025, 1:43 AM
Unknown Object (File)
Jan 30 2025, 10:05 PM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Commits
rARC9444072e2161: Improve the robustness of the "alias function" linter rule
Summary

Improve the ArcanistXHPASTLinter::LINT_ALIAS_FUNCTION linter rule. Currently this rule does not correctly handle alias functions which are not strictly lowercase.

Test Plan

Added a test case.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

joshuaspence updated this revision to Diff 31084.May 19 2015, 8:12 AM
joshuaspence retitled this revision from to Improve the robustness of the "alias function" linter rule.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptMay 19 2015, 8:12 AM
Herald added subscribers: epriestley, Korvin. · View Herald Transcript
epriestley accepted this revision.May 19 2015, 1:34 PM
epriestley edited edge metadata.
epriestley added inline comments.
src/lint/linter/ArcanistXHPASTLinter.php
4136

Sort of irrelevant since these should never be utf8, but prefer phutil_utf8_strtolower() for consistency?

This revision is now accepted and ready to land.May 19 2015, 1:34 PM
joshuaspence updated this revision to Diff 31161.May 19 2015, 10:37 PM
joshuaspence edited edge metadata.

Use phutil_utf8_strtolower

Closed by commit rARC9444072e2161: Improve the robustness of the "alias function" linter rule (authored by joshuaspence, committed by joshuaspence). · Explain WhyMay 19 2015, 10:38 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
lint/
linter/
2 lines
__tests__/
xhpast/
3 lines

Diff 31162

View Options

src/lint/linter/ArcanistXHPASTLinter.php

Loading...
View Options

src/lint/linter/__tests__/xhpast/alias-functions.lint-test

Loading...
Phabricator · Built by Phacility