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

default behavior is back to showing help without options #27

Merged
2 commits merged into from
Oct 31, 2010

Conversation

tilla
Copy link
Contributor

@tilla tilla commented Oct 31, 2010

for reading from stdin add --stdin

@RSully
Copy link

RSully commented Jan 22, 2014

Why was this and #26 closed? I would really like this functionality.

@einars
Copy link
Contributor

einars commented Jan 22, 2014

You're commenting on a three years old pull request for an extremely obsolete c++ version of the beautifier, utilizing v8 javascript engine.

@einars
Copy link
Contributor

einars commented Jan 22, 2014

(yet, I wouldn't mind if js-beautify would invoke the full help when run without parameters, instead of its quite short "Must define at least one file.\nRun js-beautify -h for help.")

@RSully
Copy link

RSully commented Jan 22, 2014

Sure, but I'd still like to see reading from stdin 😉

@einars
Copy link
Contributor

einars commented Jan 22, 2014

But you can read from stdin. js-beautify -f - or something like that.

@RSully
Copy link

RSully commented Jan 22, 2014

Oh, sorry. Wasn't clear from the help section. That works!

@einars
Copy link
Contributor

einars commented Jan 22, 2014

I myself was actually expecting "js-beautify -" to work.

@einars
Copy link
Contributor

einars commented Jan 22, 2014

...and it is weird, the js-beautify.js from the git works with '-' as a parameter, but a script installed via node doesn't.

@bitwiseman
Copy link
Member

This is ancient and closed. Please add your comment to #390.

This pull request was closed.
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.

None yet

4 participants