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

Render to STDOUT #73

Closed
joeyespo opened this issue Aug 2, 2014 · 5 comments
Closed

Render to STDOUT #73

joeyespo opened this issue Aug 2, 2014 · 5 comments
Milestone

Comments

@joeyespo
Copy link
Owner

joeyespo commented Aug 2, 2014

Currently, you can only export to a file. This makes using Grip more difficult as a general command-line tool. This is a good complement to #72.

Suggested syntax:

$ grip --export stdout

This is flexible and clear. You can still render to a file with the same name by passing ./stdout

@d13r
Copy link
Contributor

d13r commented Aug 3, 2014

The normal convention would be to use a hyphen for stdin/out: grip --export - (and grip - for #72). Though you could argue it's clearer.

@joeyespo
Copy link
Owner Author

joeyespo commented Aug 4, 2014

@davejamesmiller Yeah, that could work. That might be better since it's more conventional.

grip --export - - just looked a little odd to me. Is that the convention though?

@d13r
Copy link
Contributor

d13r commented Aug 4, 2014

Yes, I believe it is. I can't think of a standard utility where you'd use it right now, but I've used wkhtmltopdf -q - - before.

@joeyespo joeyespo added this to the 3.0 milestone Aug 7, 2014
@joeyespo
Copy link
Owner Author

joeyespo commented Aug 7, 2014

Ok cool. Just added this in 6e0e570.

Went with the more standard syntax. Thanks for weighing in, @davejamesmiller.

@joeyespo joeyespo closed this as completed Aug 7, 2014
@d13r
Copy link
Contributor

d13r commented Aug 7, 2014

👍

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

No branches or pull requests

2 participants