Page MenuHomePhabricator

Add ArcanistRaggedClassTreeEdgeXHPASTLinterRule to Phutil linter map
ClosedPublic

Authored by amckinley on Apr 24 2017, 11:13 PM.
Tags
None
Referenced Files
F19793046: D17787.id42778.diff
Feb 26 2026, 4:03 PM
F19785095: D17787.id42778.diff
Feb 24 2026, 9:22 PM
F19784310: D17787.id42778.diff
Feb 24 2026, 7:41 PM
F19784302: D17787.diff
Feb 24 2026, 7:40 PM
F19783943: D17787.id42778.diff
Feb 24 2026, 6:00 PM
F19707171: D17787.diff
Feb 11 2026, 10:51 AM
F19560669: D17787.id42778.diff
Jan 31 2026, 8:03 AM
F19555123: D17787.id42771.diff
Jan 30 2026, 8:54 AM
Subscribers

Details

Summary

Fixes T12555.

Test Plan

Added this class to the codebase and ran arc liberate:

<?php

class FooBar {
    public static function doTheFoo() {
        return 'foobar';
    }
}

Ran arc lint and observed this warning:

Warning  (XHP87) Class Not `abstract` Or `final`
 This class is neither `final` nor `abstract`, and does not have a
 docblock marking it `@concrete-extensible`.

            1 <?php
            2
 >>>        3 class FooBar {
            4     public static function doTheFoo() {
            5         return "foobar";
            6     }

Added a final modifier to FooBar's declaration and observed the warning went away.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable