@recaptime-dev's working patches + fork for Phorge, a community fork of Phabricator. (Upstream dev and stable branches are at upstream/main and upstream/stable respectively.)
hq.recaptime.dev/wiki/Phorge
phorge
phabricator
Minor fixes for `.arclint` file
Summary:
Minor fixes for the `.arclint` file. Specifically:
- Don't explicitly lint excluded paths (they are already excluded globally)
- Set the `xhpast.php-version` and `xhpast.php-version.windows` configuration (this should have been done in D9593, but wasn't)
- Sort `.arclint` linters alpahbetically
Test Plan: N/A
Reviewers: epriestley, #blessed_reviewers
Reviewed By: epriestley, #blessed_reviewers
Subscribers: epriestley, Korvin
Differential Revision: https://secure.phabricator.com/D9803
+13
-17
+13
-17
.arclint
+13
-17
.arclint
···
11
11
"filename": {
12
12
"type": "filename"
13
13
},
14
+
"generated": {
15
+
"type": "generated"
16
+
},
14
17
"javelin": {
15
18
"type": "javelin",
16
19
"include": "(\\.js$)",
17
20
"exclude": [
18
-
"(^externals/JsShrink/)",
19
-
"(^support/aphlict/)",
20
-
"(^webroot/rsrc/externals/raphael/)"
21
+
"(^support/aphlict/)"
21
22
]
22
23
},
23
24
"jshint": {
24
25
"type": "jshint",
25
26
"include": "(\\.js$)",
26
-
"exclude": [
27
-
"(^externals/JsShrink/)",
28
-
"(^webroot/rsrc/externals/raphael/)"
29
-
],
30
27
"jshint.jshintrc": "support/jshint/jshintconfig"
31
28
},
32
29
"json": {
···
39
36
"(\\.json$)"
40
37
]
41
38
},
42
-
"generated": {
43
-
"type": "generated"
44
-
},
45
39
"merge-conflict": {
46
40
"type": "merge-conflict"
47
41
},
48
42
"nolint": {
49
43
"type": "nolint"
50
44
},
45
+
"phutil-library": {
46
+
"type": "phutil-library",
47
+
"include": "(\\.php$)"
48
+
},
51
49
"phutil-xhpast": {
52
50
"type": "phutil-xhpast",
53
51
"include": "(\\.php$)",
···
55
53
"phutil_escape_html": "The phutil_escape_html() function is deprecated. Raw strings passed to phutil_tag() or hsprintf() are escaped automatically."
56
54
}
57
55
},
58
-
"phutil-library": {
59
-
"type": "phutil-library",
60
-
"include": "(\\.php$)"
56
+
"spelling": {
57
+
"type": "spelling"
61
58
},
62
59
"text": {
63
60
"type": "text",
···
74
71
"(^\\.arclint)"
75
72
]
76
73
},
77
-
"spelling": {
78
-
"type": "spelling"
79
-
},
80
74
"xhpast": {
81
75
"type": "xhpast",
82
76
"include": "(\\.php$)",
···
84
78
"16": "advice",
85
79
"34": "error"
86
80
},
87
-
"xhpast.naminghook": "PhabricatorSymbolNameLinter"
81
+
"xhpast.naminghook": "PhabricatorSymbolNameLinter",
82
+
"xhpast.php-version": "5.2.3",
83
+
"xhpast.php-version.windows": "5.3.0"
88
84
}
89
85
}
90
86
}