Skip to content
This repository has been archived by the owner on Sep 20, 2023. It is now read-only.

Switch from JSCS to Eslint #1975

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Switch from JSCS to Eslint #1975

wants to merge 1 commit into from

Conversation

azat-io
Copy link

@azat-io azat-io commented Mar 4, 2017

JSCS was merged with ESLint. Look here: http://jscs.info/

@lcd047
Copy link
Collaborator

lcd047 commented Mar 5, 2017

Right, but eslint already looks for config files in the current directory and upwards, and thus it doesn't have the problem that FAQ entry is supposed to address. Also .eslintrc is supposedly deprecated too.

Anyway, the point of that FAQ entry is to show how this kind of things can be done, rather that refer exclusively to jscs and its quirks. I suppose the jscs checker can be removed from syntastic in a future release of syntastic (say in at least a year from now), so perhaps we should postpone finding a better example until then (provided that it still makes sense by then, that is).

@ferdnyc
Copy link

ferdnyc commented Nov 3, 2019

Also .eslintrc is supposedly deprecated too.

Only in the sense that the typed .eslintrc.json, .eslintrc.js, .eslintrc.yml names are now preferred to the untyped .eslintrc name. Any JavaScript-format .eslintrc is still supported, it just needs to be named .eslintrc.js.

@lcd047
Copy link
Collaborator

lcd047 commented Nov 3, 2019

@ferdnyc Feel free to submit a patch that removes all references jscs and provides a better example in the README.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants