Changeset View
Changeset View
Standalone View
Standalone View
.arclint
Show First 20 Lines • Show All 285 Lines • ▼ Show 20 Lines | "linters": { | ||||
"(^contrib/gitian-builder/)", | "(^contrib/gitian-builder/)", | ||||
"(^contrib/apple-sdk-tools/)" | "(^contrib/apple-sdk-tools/)" | ||||
] | ] | ||||
}, | }, | ||||
"prettier": { | "prettier": { | ||||
"type": "prettier", | "type": "prettier", | ||||
"include": "(^web/.*\\.(css|html|js|json|jsx|md|scss|ts|tsx)$)", | "include": "(^web/.*\\.(css|html|js|json|jsx|md|scss|ts|tsx)$)", | ||||
"exclude": "(^web/.*/translations/.*\\.json$)" | "exclude": "(^web/.*/translations/.*\\.json$)" | ||||
}, | |||||
"eslint-regex-based": { | |||||
"type": "script-and-regex", | |||||
"include": "(\\.(jsx|js)?$)", | |||||
Fabien: This is probably not doing what you expect. I suppose you meant `(\\.jsx?$)` ? | |||||
"exclude": [], | |||||
FabienUnsubmitted Not Done Inline ActionsIf it's empty you'd better remove it Fabien: If it's empty you'd better remove it | |||||
"script-and-regex.script": "sh -c '([ -e ./web/cashtab/node_modules/.bin/eslint ]) && (./web/cashtab/node_modules/.bin/eslint -f compact \"$0\" || true)'", | |||||
FabienUnsubmitted Not Done Inline ActionsWhat is the $0 here ? There are a few reasons why I don't like script-and-regex, and here are some:
I don't see a reason why this could not be a plain PHP linter that solves all of these issues. Let me know if you need help for writing it. Fabien: What is the `$0` here ?
There are a few reasons why I don't like script-and-regex, and here… | |||||
"script-and-regex.regex": "/^(?P<file>.*): line (?P<line>[0-9]*), col (?P<char>[0-9]*), ((?P<warning>Warning)|(?P<error>Error)) - (?P<message>.*) \\((?P<code>[a-z-\\/]+)\\)$/m" | |||||
} | } | ||||
} | } | ||||
} | } |
This is probably not doing what you expect. I suppose you meant (\\.jsx?$) ?