Skip to content
This repository has been archived by the owner on Jul 24, 2024. It is now read-only.

CLI: print results to stdout by default, when no output target is supplied #669

Closed
am11 opened this issue Feb 12, 2015 · 4 comments
Closed

CLI: print results to stdout by default, when no output target is supplied #669

am11 opened this issue Feb 12, 2015 · 4 comments
Labels
Feature - Implementation Module - CLI Release - Breaking Change
Milestone
v3.0.0

Comments

@am11
Copy link
Contributor

am11 commented Feb 12, 2015

From @xzyfer's comment:

what are your thoughts on removing the --stdout and making it the default behaviour if an output directory or file isn't supplied. This the behaviour of Ruby sass and Libsass.

Adding to v2.1 milestone.
@am11 am11 added the v2.1 label Feb 12, 2015
@am11 am11 added this to the 2.1 milestone Feb 12, 2015
am11 added a commit to am11/node-sass that referenced this issue Feb 23, 2015
@am11
CLI: Retires stdout option.
1dc8ab7 
When no destination or output folder is provided,
it will print the output to stdout.

If the source-map option was set, it will print
it as well.

Issue URL: sass#669.
PR URL: sass#701.
@am11 am11 mentioned this issue Feb 23, 2015
Merged
am11 added a commit to am11/node-sass that referenced this issue Feb 23, 2015
@am11
CLI: Retires stdout option.
20d666f 
When no destination or output folder is provided,
it will print the output to stdout.

If the source-map option was set, it will print
it as well.

Issue URL: sass#669.
PR URL: sass#701.
am11 added a commit to am11/node-sass that referenced this issue Feb 23, 2015
@am11
CLI: Retires stdout option.
97cae50 
When no destination or output folder is provided,
it will print the output to stdout.

If the source-map option was set, it will print
it as well.

Issue URL: sass#669.
PR URL: sass#701.
am11 added a commit to am11/node-sass that referenced this issue Feb 23, 2015
@am11
CLI: Retires stdout option.
d140077 
When no destination or output folder is provided,
it will print the output to stdout.

If the source-map option was set, it will print
it as well.

Issue URL: sass#669.
PR URL: sass#701.
am11 added a commit to am11/node-sass that referenced this issue Feb 23, 2015
@am11
CLI: Retires stdout option.
433e939 
When no destination or output folder is provided,
it will print the output to stdout.

If the source-map option was set, it will print
it as well.

Issue URL: sass#669.
PR URL: sass#701.
@am11
Copy link
Contributor Author

am11 commented Feb 28, 2015

Addressed by 433e939 via #701.

@am11 am11 closed this as completed Feb 28, 2015
@qm3ster qm3ster mentioned this issue Jan 30, 2016
Merged
@qm3ster
Copy link
Contributor

qm3ster commented Jan 30, 2016

README.md still says

Output will be saved with the same name as input Sass file into the current working directory if the --output flag is omitted.

Fixed in #1363 please merge.

@am11
Copy link
Contributor Author

am11 commented Jan 30, 2016

@qm3ster, merged, thanks!
Aside, note that --output accepts the directory path instead of path including filename. So node-sass blah.scss --output myblah.css will actually write to {cwd}/myblah.css/blah.css (it will also do the mkdir myblah, if that was non-existent). If the intent is to write to a file with different name, then we can redirect output as node-sass blah.scss > myblah.css.

@qm3ster
Copy link
Contributor

qm3ster commented Jan 31, 2016

Yup, well aware, I use both successfully.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
Feature - Implementation Module - CLI Release - Breaking Change
Projects
None yet
Milestone
v3.0.0
Development

No branches or pull requests
2 participants
@am11 @qm3ster