Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update plan #13

Closed
poppinlp opened this issue Feb 22, 2018 · 4 comments
Closed

Update plan #13

poppinlp opened this issue Feb 22, 2018 · 4 comments

Comments

@poppinlp
Copy link
Contributor

Hi. I think there could be some updates. Here's my plan:

  • Add help information for CLI
  • Support more configuration file formats such as yaml (I have seen .yaml-lint.json in code but this not mentioned in docs. We may update docs)
  • Add test cases for CLI since we only have test cases for linter right now.

I don't know what you think, so I haven't started this yet.
Wait for your reply~ Wish not bother you too much. :)

@rasshofer
Copy link
Owner

First of all, thank you very much for your ongoing effort and dedication, highly appreciated!

Regarding your ideas/plans: I already added some documentation regarding how to specify options with 547f5e0, but if you like you could implement the test cases for the CLI, that would be amazing! :)

@poppinlp
Copy link
Contributor Author

poppinlp commented Mar 5, 2018

OK, i'll add the test cases for the CLI.
And maybe you could active this repo on coveralls or something like it, so that we can update the code coverage by CI and show it by badge. (I don't have the permission to do this)

@rasshofer
Copy link
Owner

Coveralls has already been enabled: https://coveralls.io/github/rasshofer/yaml-lint :)

Coverage Status

@poppinlp
Copy link
Contributor Author

poppinlp commented Mar 6, 2018

Hi, i've add test cases in this PR.

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

No branches or pull requests

2 participants