Skip to content
This repository has been archived by the owner on Jan 19, 2019. It is now read-only.

Docs: add options to README #530

Merged
merged 2 commits into from
Nov 5, 2018
Merged

Docs: add options to README #530

merged 2 commits into from
Nov 5, 2018

Conversation

trusktr
Copy link
Contributor

@trusktr trusktr commented Oct 25, 2018

Are these options inherited from somewhere? If so, then maybe we should just mention that, and only list ones that are specific to this parser.

@jsf-clabot
Copy link

jsf-clabot commented Oct 25, 2018

CLA assistant check
All committers have signed the CLA.

Copy link
Member

@kaicataldo kaicataldo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for contributing! The underlying parser this package uses was recently separated out into its own package (called typescript-estree), and that's where the option is actually used.

Do you mind signing the CLA so we can merge this?

@trusktr
Copy link
Contributor Author

trusktr commented Nov 5, 2018

I signed the CLA. I also updated the README. How does that look?

Copy link
Member

@kaicataldo kaicataldo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks!

@kaicataldo
Copy link
Member

Ah, looks like the commit message doesn't match our guidelines. I'll update it, but for posterity, more information can be found here.

@kaicataldo kaicataldo changed the title add options to README Docs: add options to README Nov 5, 2018
@kaicataldo kaicataldo merged commit 6a9b629 into eslint:master Nov 5, 2018
@kaicataldo
Copy link
Member

Thanks for contributing!

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

Successfully merging this pull request may close these issues.

3 participants