Skip to content
This repository has been archived by the owner on Mar 25, 2021. It is now read-only.

Support yaml configuration #1598

Closed
remcohaszing opened this issue Oct 5, 2016 · 5 comments
Closed

Support yaml configuration #1598

remcohaszing opened this issue Oct 5, 2016 · 5 comments
Labels
P3 Status: Accepting PRs Type: Feature Request

Comments

@remcohaszing
Copy link

It would be nice if the configuration file could be written in yaml format instead of json.

why? Unlike json, yaml supports comments.

why? Both eslint and stylelint support this. It's nice if linters support similar configuration files.
@adidahiya
Copy link
Contributor

yeah, that'd be nice. stylelint uses cosmiconfig, which I'm tracking as an enhancement here #1071. accepting PRs :)

@jkillian
Copy link
Contributor

jkillian commented Oct 5, 2016 •
edited

@remcohaszing We actually support comments in our configuration JSON files. Yes, technically outside of the standard, but it's useful! 😄

@adidahiya adidahiya added P3 and removed P2 labels Oct 5, 2016
@remcohaszing
Copy link
Author

JSON comments help a lot! 😄 However, I'd still prefer yaml support. 😉

@nchen63 nchen63 mentioned this issue Nov 15, 2016
Closed
@jsoendermann jsoendermann mentioned this issue Mar 21, 2017
Closed
4 tasks
@hronro
Copy link

hronro commented Aug 1, 2017

+1

jhanschoo added a commit to jhanschoo/tslint that referenced this issue Nov 1, 2017
@jhanschoo
Support yaml configuration (palantir#1598)
8233637
@jhanschoo jhanschoo mentioned this issue Nov 1, 2017
Merged
4 tasks
jhanschoo added a commit to jhanschoo/tslint that referenced this issue Nov 1, 2017
@jhanschoo
Support yaml configuration (palantir#1598)
833ccc7
jhanschoo added a commit to jhanschoo/tslint that referenced this issue Nov 1, 2017
@jhanschoo
Support yaml configuration (palantir#1598)
dcf5c55
jhanschoo added a commit to jhanschoo/tslint that referenced this issue Nov 5, 2017
@jhanschoo
incorporate feedback from @ajafff on issue (palantir#1598)
f786ea1
jhanschoo added a commit to jhanschoo/tslint that referenced this issue Nov 6, 2017
@jhanschoo
incorporate feedback from @ajafff on issue (palantir#1598)
71504fc
jhanschoo added a commit to jhanschoo/tslint that referenced this issue Nov 6, 2017
@jhanschoo
incorporate feedback pt. 2 from @ajafff on issue (palantir#1598)
e3bb4c5
jhanschoo added a commit to jhanschoo/tslint that referenced this issue Nov 7, 2017
@jhanschoo
incorporate feedback from @adidahiya on PR for issue (palantir#1598)
fdaa6db
ajafff pushed a commit that referenced this issue Nov 8, 2017
@jhanschoo @ajafff
Support yaml configuration (#1598) (#3433)
6d3f78e
@ajafff
Copy link
Contributor

ajafff commented Nov 8, 2017

Fixed by #3433

@ajafff ajafff closed this as completed Nov 8, 2017
HyphnKnight pushed a commit to HyphnKnight/tslint that referenced this issue Apr 9, 2018
@jhanschoo @HyphnKnight
Support yaml configuration (palantir#1598) (palantir#3433)
adb5ac9
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
P3 Status: Accepting PRs Type: Feature Request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@adidahiya @remcohaszing @jkillian @ajafff @hronro