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

[DOCS] Improve documentation #50

Closed
TomCockram opened this issue Apr 15, 2021 · 2 comments
Closed

[DOCS] Improve documentation #50

TomCockram opened this issue Apr 15, 2021 · 2 comments
Labels
enhancement New feature or request

Comments

@TomCockram
Copy link

TomCockram commented Apr 15, 2021

What would you like to change about the program?
I believe the documentation needs updating or improving as I tried to use the options below and always got the message that they were unknown options?

Providing some examples of how you're meant to use these options would help. Also showing an action that uses more than one option at a time would show how to chain them in case you have to do something different.

How are you meant to use the options like --dry and --only_changed?

[warn] Ignored unknown option --dry=true.
[warn] Ignored unknown option --only_changed=true. 
[warn] Ignored unknown option --dry.
[warn] Ignored unknown option --only_changed.

Have tried both of these and always complained about unknown options.

Why do you think this is a cool idea?
The documentation needs to be more clear on the examples and how you're meant to use them. Also, I believe there is something that is required but is not mentioned as required.

Error: No parser and no file path given, couldn't infer a parser.

I get this error but no idea how to fix or what I'm missing.

Happy to update the readme file if I have the correct information!

@TomCockram TomCockram added the enhancement New feature or request label Apr 15, 2021
@TomCockram TomCockram changed the title [BUG] Improve documentation Improve documentation Apr 15, 2021
@TomCockram TomCockram changed the title Improve documentation [DOCS] Improve documentation Apr 15, 2021
@creyD
Copy link
Owner

creyD commented Apr 23, 2021

You may create a PR and I will merge if it is beneficial. Thanks!

@creyD
Copy link
Owner

creyD commented Jul 15, 2021

Also both of these options will need to be used in your .yml file under the with: section of the configuration:

with:
# This part is also where you can pass other options, for example:
prettier_options: --write **/*.{js,md}
dry: True

The second example actually shows this for only-changed, which is why I am closing this. If you have other concerns, please reopen!

@creyD creyD closed this as completed Jul 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants