Skip to content

Conversation

@zgreen
Copy link
Contributor

@zgreen zgreen commented May 25, 2017

It wasn't clear to me whether I could pass a directory of files as an input—it turns out I can! This PR adds an example command to the README, showing how to use a glob pattern as the input for the CLI.

This is a terrific tool, thanks!

It wasn't clear to me whether I could pass a directory of files as an input—it turns out I can! This PR adds an example command to the README, showing how to use a glob pattern as the input for the CLI.

This is a terrific tool, thanks!
Copy link
Collaborator

@RyanZim RyanZim left a comment

Choose a reason for hiding this comment

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

Good idea, but I don't want the big heading. Put this example right after the first one with the text

Also accepts globs

or something like that.

Also, if you want to recursively read a directory, do postcss src/**/*.css.

Perhaps rename the directory to src or css?

- use blockquotes
- add a recursive read example
@zgreen
Copy link
Contributor Author

zgreen commented May 25, 2017

Thanks, @RyanZim. Updated docs per your suggestions.

Copy link
Collaborator

@RyanZim RyanZim left a comment

Choose a reason for hiding this comment

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

Sorry to be so picky 😉

README.md Outdated
postcss [input.css] [OPTIONS] [-o|--output output.css] [-w|--watch]
```

> The input may also be a glob:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Just use normal text, not a blockquote. (No >)

README.md Outdated
postcss src/*.css [OPTIONS]
```

> Recursively read a directory:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Ditto

@zgreen
Copy link
Contributor Author

zgreen commented May 25, 2017

Np! Updated again.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 95.89% when pulling 2cddf24 on zgreen:patch-1 into dfc68e7 on postcss:master.

@RyanZim RyanZim merged commit cc78d46 into postcss:master May 25, 2017
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

Successfully merging this pull request may close these issues.

3 participants