Diffusion Arcanist (legacy-2019) History

rARC/src/lint/linter
legacy-2019

Oct 17 2019

da6d4f85eefb Add a lint check for deprecated argument order to "implode()"
Author
epriestley
Revisions
D20857: Add a lint check for deprecated argument order to "implode()"
Browse Repository
Passed
No Audits

May 21 2019

dd514e268b2c Modify the `lint-test` file format to allow for more powerful assertions
Author
joshuaspence
Revisions
D11176: Modify the `lint-test` file format to allow for more powerful assertions
Browse Repository
Passed
No Audits

May 15 2019

82445bb60534 Let lint rules support anonymous classes
Author
wjiang
Browse Repository
Passed
No Audits

May 14 2019

6a8e76db329e Allow `buildFutures` and `resolveFutures` to be overridden
Author
joshuaspence
Browse Repository
Passed
No Audits

fceac878f112 Allow `setCustomSeverityRules` to be overridden in subclasses
Author
joshuaspence
Browse Repository
Passed
No Audits

4f583dded366 Call `$linter->setEngine` in linter tests
Author
joshuaspence
Browse Repository
Passed
No Audits

b61e890a0ca8 Remove unnecessary "," from Pylint version regex
Author
epriestley
Browse Repository
Passed
No Audits

Feb 12 2019

7e61e43f6554 Add version check whitelists for constants to the version compatibility lint…
Author
epriestley
Revisions
D20146: Add version check whitelists for constants to the version compatibility lint rule
Browse Repository
Passed
No Audits

Dec 28 2018

25c2381959ac Statically detect "continue" inside "switch"
Author
epriestley
Revisions
D19931: Statically detect "continue" inside "switch"
Browse Repository
Passed
No Audits

Dec 12 2018

eb732555a71a Fix a "continue;" inside a switch in ArcanistPHPCompatibilityXHPASTLinterRule
Author
epriestley
Revisions
D19868: Fix a "continue;" inside a switch in ArcanistPHPCompatibilityXHPASTLinterRule
Browse Repository
Passed
No Audits

Sep 6 2018

30b7835c37b5 Allow `willLintPaths` and `didLintPaths` to be overridden
Author
joshuaspence
Revisions
D19630: Allow `willLintPaths` and `didLintPaths` to be overridden
Browse Repository
Passed
No Audits

Aug 3 2018

d9a4293ae734 Consolidate redundant "should should" from some linter help strings in Arcanist
Author
epriestley
Revisions
D19561: Consolidate redundant "should should" from some linter help strings in Arcanist
Browse Repository
Passed
No Audits

Sep 20 2017

b836557b12b3 Correct lint rendering when patching trailing whitespace in files
Author
epriestley
Revisions
D18631: Correct lint rendering when patching trailing whitespace in files
Browse Repository
Passed
No Audits

Aug 31 2017

be67df611856 Extract and cover the logic for "trimming" a lint message
Author
epriestley
Revisions
D18509: Extract and cover the logic for "trimming" a lint message
Browse Repository
Passed
No Audits

Jul 22 2017

836768bdccd4 Fix ArcanistPHPCloseTagXHPASTLinterRule always bailing out
Author
asherkin
Revisions
D18271: Fix ArcanistPHPCloseTagXHPASTLinterRule always bailing out
Browse Repository
Passed
No Audits

May 3 2017

3c4735795a29 Detect trailing spaces and tabs
Author
johnny-bit
Revisions
D17814: Detect trailing spaces and tabs
Browse Repository
Passed
No Audits

Apr 25 2017

5d0f5afca8cd Add ArcanistRaggedClassTreeEdgeXHPASTLinterRule to Phutil linter map
Author
amckinley
Revisions
D17787: Add ArcanistRaggedClassTreeEdgeXHPASTLinterRule to Phutil linter map
Browse Repository
Passed
No Audits

Mar 24 2017

d1db9a72b552 Improve the flake8 line-matching regex
Author
jparise
Revisions
D17552: Improve the flake8 line-matching regex
Browse Repository
Passed
No Audits

Feb 18 2017

3b6b523c2b23 Fix errors found by PHPStan
Author
vrana
Revisions
D17376: Fix errors found by PHPStan
Browse Repository
Passed
No Audits

Feb 16 2017

d0353d238113 Fix errors found by PHPStan
Author
vrana
Revisions
D17367: Fix errors found by PHPStan
Browse Repository
Passed
No Audits

Aug 1 2016

f20d4b15c7e4 Add a lint error about use of PHP short array syntax ('[...]')
Author
epriestley
Revisions
D16357: Add a lint error about use of PHP short array syntax ('[...]')
Browse Repository
Passed
No Audits

Apr 29 2016

c58f1b9a2507 Prefer pip to easy_install
Author
eadler
Revisions
D15818: Prefer pip to easy_install
Browse Repository
Passed
No Audits

768e1a56bc5f When running XHPAST unit tests, include the "syntax error" lint rule
Author
epriestley
Revisions
D15819: When running XHPAST unit tests, include the "syntax error" lint rule
Browse Repository
Passed
No Audits

3ffed59bd7fb Update xhpast linter rules for new function AST format
Author
richardvanvelzen
Revisions
D15814: Update xhpast linter rules for new function AST format
Browse Repository
Passed
No Audits

ea6fc7789295 Fix incorrect variable for linter standards
Author
joshuaspence
Revisions
D15212: Fix incorrect variable for linter standards
Browse Repository
Passed
No Audits

Apr 27 2016

9b07d1c480f4 Recognize error codes from Flake8 extensions
Author
jparise
Revisions
D15810: Recognize error codes from Flake8 extensions
Browse Repository
Passed
No Audits

Jan 15 2016

b87138356a9c Cleans up some minor issues in cpplint
Author
michaeloa
Revisions
D15030: Cleans up some minor issues in cpplint
Browse Repository
Passed
No Audits

Jan 14 2016

22af67c1c05e Minor fixes to arcanist cpplint
Author
michaeloa
Revisions
D15019: Minor fixes to arcanist cpplint
Browse Repository
Passed
No Audits

Jan 12 2016

05c12eb9d9cb If the Script-and-Regex linter captures no "line" text, treat the message as…
Author
epriestley
Revisions
D15000: If the Script-and-Regex linter captures no "line" text, treat the message as affecting the entire file
Browse Repository
Passed
No Audits

Jan 6 2016

aeb374b33348 Fix cpplint message regex
Author
jra3
Revisions
D14960: Fix cpplint message regex
Browse Repository
Passed
No Audits

Dec 22 2015

b3e68c9f1793 Add a linter rule for use of `is_a`
Author
joshuaspence
Revisions
D14573: Add a linter rule for use of `is_a`
Browse Repository
Passed
No Audits

9ee14d28490c Improve the "self member reference" linter rule
Author
joshuaspence
Revisions
D13935: Improve the "self member reference" linter rule
Browse Repository
Passed
No Audits

Dec 8 2015

0c8124a2729d Fix handling of empty array index
Author
joshuaspence
Browse Repository
Passed
Audited

d2e778549727 Add a linter rule for curly brace array indexes
Author
joshuaspence
Revisions
D14603: Add a linter rule for curly brace array indexes
Browse Repository
Passed
No Audits

Dec 7 2015

a6e81daad1bb Improve brace formatting linter rule
Author
joshuaspence
Revisions
D14690: Improve brace formatting linter rule
Browse Repository
Passed
No Audits

b52e9dc702c2 Linter fixes
Author
joshuaspence
Revisions
D14633: Linter fixes
Browse Repository
Passed
No Audits

Dec 2 2015

cdaad096fa8d Add a linter rule for public properties
Author
joshuaspence
Revisions
D14641: Add a linter rule for public properties
Browse Repository
Passed
No Audits

3c193984da6f Add a "binary integer casing" linter rule
Author
joshuaspence
Revisions
D14644: Add a "binary integer casing" linter rule
Browse Repository
Passed
No Audits

ae3c2cb0e172 Add binary integers to `ArcanistPHPCompatibilityXHPASTLinterRule`
Author
joshuaspence
Revisions
D14643: Add binary integers to `ArcanistPHPCompatibilityXHPASTLinterRule`
Browse Repository
Passed
No Audits

560e4ae4919b Add a linter rule for invalid octals
Author
joshuaspence
Revisions
D14604: Add a linter rule for invalid octals
Browse Repository
Passed
No Audits

8ed3c45c8226 Use `CaseInsensitiveArray` in `ArcanistFunctionCallShouldBeTypeCastXHPASTLinter…
Author
joshuaspence
Revisions
D14625: Use `CaseInsensitiveArray` in `ArcanistFunctionCallShouldBeTypeCastXHPASTLinterRule`
Browse Repository
Passed
No Audits

76ae02325dd3 Fix another edge case for "function call should be type cast" linter rule
Author
joshuaspence
Revisions
D14623: Fix another edge case for "function call should be type cast" linter rule
Browse Repository
Passed
No Audits

09052d52470d Minor fix for typecast linter rule
Author
joshuaspence
Revisions
D14611: Minor fix for typecast linter rule
Browse Repository
Passed
No Audits

Dec 1 2015

00efcd294fd0 Add a linter rule for hexadecimal number casing
Author
joshuaspence
Revisions
D14612: Add a linter rule for hexadecimal number casing
Browse Repository
Passed
No Audits

5218ec357c5c Add a table showing all XHPAST linter rules to the output of `arc linters…
Author
joshuaspence
Revisions
D14563: Add a table showing all XHPAST linter rules to the output of `arc linters xhpast`
Browse Repository
Passed
No Audits

Nov 30 2015

87306cfe1453 Add a linter rule for variable reference spacing
Author
joshuaspence
Revisions
D14602: Add a linter rule for variable reference spacing
Browse Repository
Passed
No Audits

Nov 29 2015

578f540a92b1 Add a linter rule for `*val` functions
Author
joshuaspence
Revisions
D14572: Add a linter rule for `*val` functions
Browse Repository
Passed
No Audits

Nov 25 2015

eeaa176cfc5a Add a linter rule to ensure that `namespace` is the first statement in a file
Author
joshuaspence
Revisions
D14526: Add a linter rule to ensure that `namespace` is the first statement in a file
Browse Repository
Passed
No Audits

600dcf83dd11 Fix linter rule ID
Author
joshuaspence
Browse Repository
Passed
Audited

b323ad4d6484 Add a linter rule for `abstract` methods within an `interface`
Author
joshuaspence
Revisions
D14562: Add a linter rule for `abstract` methods within an `interface`
Browse Repository
Failed
No Audits

8183a458048c Add a linter rule for `interface` method bodies
Author
joshuaspence
Revisions
D14561: Add a linter rule for `interface` method bodies
Browse Repository
Passed
No Audits

28f5b0ddc6bb Fix a compatibility issue with `ArcanistUnsafeDynamicStringXHPASTLinterRule`
Author
joshuaspence
Revisions
D14567: Fix a compatibility issue with `ArcanistUnsafeDynamicStringXHPASTLinterRule`
Browse Repository
Passed
No Audits

Nov 23 2015

9e78d15fc0c7 Improve the "unexpected return value" linter rule
Author
joshuaspence
Revisions
D14564: Improve the "unexpected return value" linter rule
Browse Repository
Passed
No Audits

72d9013d29dc Add a linter rule for `abstract` methods containing a body
Author
joshuaspence
Revisions
D14560: Add a linter rule for `abstract` methods containing a body
Browse Repository
Passed
No Audits

37571d883952 Add a linter rule to determine whether a class should be marked as `abstract`
Author
joshuaspence
Revisions
D14559: Add a linter rule to determine whether a class should be marked as `abstract`
Browse Repository
Passed
No Audits

20e99acf0ad1 Add a linter rule for `abstract private` methods
Author
joshuaspence
Revisions
D14558: Add a linter rule for `abstract private` methods
Browse Repository
Passed
No Audits

6f908f633b7f Add a linter rule for unnecessary symbol aliases
Author
joshuaspence
Revisions
D14557: Add a linter rule for unnecessary symbol aliases
Browse Repository
Passed
No Audits

3b41e62f8700 Use Remarkup in linter messages
Author
joshuaspence
Revisions
D14489: Use Remarkup in linter messages
Browse Repository
Passed
No Audits

ae210fda9f4b Add a linter rule for `use` statement namespace prefixes
Author
joshuaspence
Revisions
D14517: Add a linter rule for `use` statement namespace prefixes
Browse Repository
Passed
No Audits

Nov 19 2015

2eada58960e5 Fix cppcheck linter test case
Author
joshuaspence
Revisions
D14524: Fix cppcheck linter test case
Browse Repository
Passed
No Audits

187e10e7810b Fix parsing of `ruby` version for `ArcanistRubyLinter`
Author
joshuaspence
Revisions
D14525: Fix parsing of `ruby` version for `ArcanistRubyLinter`
Browse Repository
Passed
No Audits

581829a99ee8 Write a linter rule for `$this` reassignment
Author
joshuaspence
Revisions
D14514: Write a linter rule for `$this` reassignment
Browse Repository
Passed
No Audits

426d535b13d7 Fix PHP compatibility linter after improvements to parsing of `use` statements
Author
joshuaspence
Revisions
D14519: Fix PHP compatibility linter after improvements to parsing of `use` statements
Browse Repository
Passed
No Audits

3ac313ad1e65 Write a linter rule for unexpected `return` values
Author
joshuaspence
Revisions
D14516: Write a linter rule for unexpected `return` values
Browse Repository
Passed
No Audits

149e7895fbd5 Add a linter rule for nested namespaces
Author
joshuaspence
Revisions
D14513: Add a linter rule for nested namespaces
Browse Repository
Passed
No Audits

Nov 18 2015

7386afc953e0 Add a linter rule for `parent` references
Author
joshuaspence
Revisions
D14443: Add a linter rule for `parent` references
Browse Repository
Passed
No Audits

4d512c51d43c Test XHPAST linter rules in isolation
Author
joshuaspence
Revisions
D14010: Test XHPAST linter rules in isolation
Browse Repository
Passed
No Audits

Nov 17 2015

e3e232530c6e Fix brace formatting linter rule after XHPAST changes
Author
joshuaspence
Revisions
D14499: Fix brace formatting linter rule after XHPAST changes
Browse Repository
Passed
No Audits

b25cf080bc55 Fix ArcanistCSSLintLinter issue for messages without line number
Author
dereckson
Revisions
D14497: Fix ArcanistCSSLintLinter issue for messages without line number
Browse Repository
Passed
No Audits

Nov 12 2015

9dd6eafb5254 Fold `ArcanistPhutilXHPASTLinter` into `ArcanistXHPASTLinter`
Author
joshuaspence
Revisions
D13867: Fold `ArcanistPhutilXHPASTLinter` into `ArcanistXHPASTLinter`
Browse Repository
Passed
No Audits

dd0deb240700 Add XHAST linter standards
Author
joshuaspence
Revisions
D13942: Add XHAST linter standards
Browse Repository
Passed
No Audits

Nov 2 2015

d12bcdc68394 Remove some unused methods from ArcanistLinter
Author
joshuaspence
Revisions
D14065: Remove some unused methods from ArcanistLinter
Browse Repository
Passed
No Audits

Oct 21 2015

b3ea439f4d86 External linters can now specify a version requirement.
Author
jparise
Revisions
D14298: External linters can now specify a version requirement.
Browse Repository
Passed
No Audits

Oct 19 2015

e51e1c3d4464 Allow Script-and-Regex linter to have optional/empty capturing patterns for…
Author
epriestley
Revisions
D14307: Allow Script-and-Regex linter to have optional/empty capturing patterns for char/line
Browse Repository
Passed
No Audits

Oct 2 2015

1b4a3e0c5eb9 Set path on more linters
Author
vm
Revisions
D14225: Set name on more linters
Browse Repository
Failed
No Audits

f0c57986bc32 Fix ArcanistClosureLinter
Author
vm
Revisions
D14220: Fix ArcanistClosureLinter
Browse Repository
Failed
No Audits

Sep 25 2015

e8a0ebaeffaa Improve validation of 'name' and 'code' parameters for lint messages
Author
epriestley
Revisions
D14165: Improve validation of 'name' and 'code' parameters for lint messages
Browse Repository
No Builds
No Audits

Sep 14 2015

501007f0122e More linter's short description
Author
avivey
Revisions
D14106: More linter's short description
Browse Repository
No Builds
No Audits

Sep 8 2015

28b89785fe5f update linter short desc
Author
avivey
Revisions
D14085: update linter short desc
Browse Repository
No Builds
No Audits

Sep 2 2015

9419cccdd2b1 Delete another problematic XML test file
Author
joshuaspence
Revisions
D14027: Delete another problematic XML test file
Browse Repository
No Builds
No Audits

Sep 1 2015

6fa2de5ff813 Add a linter rule for detecting empty files
Author
joshuaspence
Revisions
D13881: Add a linter rule for detecting empty files
Browse Repository
No Builds
No Audits

Aug 30 2015

a5304e472d18 Add a linter rule for newlines after PHP open tags
Author
joshuaspence
Revisions
D13534: Add a linter rule for newlines after PHP open tags
Browse Repository
No Builds
No Audits

Aug 27 2015

18e32d6ec7f7 Fix linter rule after XHPAST change
Author
joshuaspence
Revisions
D13961: Fix linter rule after XHPAST change
Browse Repository
No Builds
No Audits

Aug 25 2015

43f8e7eb7119 Recover from PyLint raising messages at character "-1"
Author
epriestley
Revisions
D13991: Recover from PyLint raising messages at character "-1"
Browse Repository
No Builds
No Audits

1d3266395f70 Remove two problematic XML linter unit tests
Author
epriestley
Revisions
D13992: Remove two problematic XML linter unit tests
Browse Repository
No Builds
No Audits

Aug 20 2015

9b8c9d280ea4 Exclude variables used in strings inside closures when checking for undeclared…
Author
joshuaspence
Revisions
D13938: Exclude variables used in strings inside closures when checking for undeclared variables
Browse Repository
No Builds
No Audits

Aug 19 2015

05aaa1a5a3b8 ArcanistPyLintLinter fix to getMandatoryFlags msg-template
Author
ealbright <ealbright@terrapower.com>
Revisions
D13931: ArcanistPyLintLinter fix to getMandatoryFlags msg-template
Browse Repository
No Builds
Audited

f9bd6b058f8a Add a Composer linter
Author
joshuaspence
Revisions
D13883: Add a Composer linter
Browse Repository
No Builds
Audited

27ec3a2e34aa Improve linter handling of short array syntax
Author
joshuaspence
Revisions
D13895: Improve linter handling of short array syntax
Browse Repository
No Builds
Audited

4404e66735c4 Improve the "inline HTML" linter rule
Author
joshuaspence
Revisions
D13896: Improve the "inline HTML" linter rule
Browse Repository
No Builds
Audited

Aug 13 2015

fe8ed2a6f8b0 Add a linter rule for the use of `parse_str`
Author
joshuaspence
Revisions
D13857: Add a linter rule for the use of `parse_str`
Browse Repository
No Builds
No Audits

2e76e2965cbe Add a linter rule for inline HTML
Author
joshuaspence
Revisions
D13871: Add a linter rule for inline HTML
Browse Repository
No Builds
No Audits

bf88f4616cdf Add a linter rule for global variables
Author
joshuaspence
Revisions
D13882: Add a linter rule for global variables
Browse Repository
No Builds
No Audits

Aug 12 2015

f4c322cb72cd Add a linter rule for list assignments
Author
joshuaspence
Revisions
D13870: Add a linter rule for list assignments
Browse Repository
No Builds
No Audits

Aug 11 2015

423e7d238933 Move a test file
Author
joshuaspence
Revisions
D13866: Move a test file
Browse Repository
No Builds
No Audits

e00e2939c164 Various linter fixes
Author
joshuaspence
Revisions
D13861: Various linter fixes
Browse Repository
No Builds
No Audits

31a49196801a Improve linter rules for array formatting
Author
joshuaspence
Revisions
D13859: Improve linter rules for array formatting
Browse Repository
No Builds
No Audits

b82e4cd40e1f Fix failing JSHint linter test
Author
joshuaspence
Revisions
D13860: Fix failing JSHint linter test
Browse Repository
No Builds
No Audits

Aug 10 2015

504ff4268188 Minor improvement for array value linter rule
Author
joshuaspence
Revisions
D13856: Minor improvement for array value linter rule
Browse Repository
No Builds
No Audits

830bcbc2a5e9 Add a linter rule for array elements
Author
joshuaspence
Revisions
D13842: Add a linter rule for array elements
Browse Repository
No Builds
No Audits