Page MenuHomePhabricator

Remove the noconfig flag
ClosedPublic

Authored by jjoos on Jan 6 2015, 12:58 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Apr 27, 8:32 PM
Unknown Object (File)
Mon, Apr 22, 7:34 PM
Unknown Object (File)
Mar 28 2024, 9:35 PM
Unknown Object (File)
Mar 28 2024, 9:35 PM
Unknown Object (File)
Mar 28 2024, 8:53 PM
Unknown Object (File)
Mar 28 2024, 2:03 PM
Unknown Object (File)
Mar 25 2024, 11:44 AM
Unknown Object (File)
Mar 25 2024, 11:44 AM
Subscribers
Tokens
"Like" token, awarded by tomdev.

Details

Summary

D11196 introduced a regression that made it impossible for coffeelint to read any configuration. I'm reverting the change in that diff.

Created a pull request changing the documentation of coffeelint to make sure I'm interpreting this flag correctly: https://github.com/clutchski/coffeelint/pull/364

Test Plan
  • patch
  • npm install -g coffeelint
  • create a test.coffee file with:
# 1234567890
  • arc lint test.coffee
OKAY  No lint warnings.
  • create a coffeelint.json with
{
  "max_line_length": {
    "value": 10
  }
}
  • arc lint test.coffee, expected output:
>>> Lint for test.coffee:

   Error  (COFFEE)
    Line exceeds maximum allowed length.

    >>>        1 # 1234567890
  • create a .arclint with
{
  "linters": {
    "coffeelint": {
      "type": "coffeelint",
      "coffeelint.config": "coffee_lint_config_with_different_name.json"
    }
  }
}
  • rename coffeelint.json to coffee_lint_config_with_different_name.json
  • arc lint test.coffee, expected output:
>>> Lint for test.coffee:

   Error  (COFFEE)
    Line exceeds maximum allowed length.

    >>>        1 # 1234567890

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

jjoos retitled this revision from to Remove the noconfig flag.
jjoos updated this object.
jjoos edited the test plan for this revision. (Show Details)
jjoos edited edge metadata.
jjoos edited the test plan for this revision. (Show Details)

It is confirmed that this was a bug in the coffeelint docs: https://github.com/clutchski/coffeelint/pull/364

This revision is now accepted and ready to land.Jan 6 2015, 2:59 PM
This revision was automatically updated to reflect the committed changes.