Page MenuHomePhabricator

Remove the noconfig flag
ClosedPublic

Authored by jjoos on Jan 6 2015, 12:58 PM.
Tags
None
Referenced Files
F13207883: D11250.id27004.diff
Thu, May 16, 5:41 AM
F13199241: D11250.diff
Mon, May 13, 1:01 PM
F13190773: D11250.id27004.diff
Sat, May 11, 1:32 PM
Unknown Object (File)
Sat, May 4, 1:11 PM
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
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
Branch
enable-coffee-linter-to-read-config
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 3633
Build 3642: [Placeholder Plan] Wait for 30 Seconds

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.