Page MenuHomePhabricator

Present case of class and interface names.
ClosedPublic

Authored by joshuaspence on Jun 11 2014, 5:46 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, May 7, 5:29 AM
Unknown Object (File)
Fri, May 3, 3:20 AM
Unknown Object (File)
Mon, Apr 29, 2:38 PM
Unknown Object (File)
Wed, Apr 24, 10:11 PM
Unknown Object (File)
Thu, Apr 11, 8:09 AM
Unknown Object (File)
Feb 19 2024, 2:07 PM
Unknown Object (File)
Feb 19 2024, 5:25 AM
Unknown Object (File)
Feb 15 2024, 11:11 PM
Subscribers

Details

Summary

Currently, class and interface names are converted to lowercase when comparing minimum and maximum versions. This is necessary in order to compare with the data from php_compat_info.json but causes the lint message to be slightly misleading.

Test Plan

Linted a test file.

Before

Error  (XHP31) Use Of PHP 5.3 Features
 This codebase targets PHP 5.2.3, but `iterator` was not introduced until
 PHP 5.3.0.

            1 <?php
            2
 >>>        3 final class FooBar implements Iterator {}

After

Error  (XHP31) Use Of PHP 5.3 Features
 This codebase targets PHP 5.2.3, but `Iterator` was not introduced until
 PHP 5.3.0.

            1 <?php
            2
 >>>        3 final class FooBar implements Iterator {}

Diff Detail

Repository
rARC Arcanist
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

joshuaspence retitled this revision from to Present case of class and interface names..
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
epriestley edited edge metadata.
This revision is now accepted and ready to land.Jun 11 2014, 12:39 PM
epriestley updated this revision to Diff 22647.

Closed by commit rARCb189b594d20c (authored by @joshuaspence, committed by @epriestley).