Skip to content
This repository has been archived by the owner on Aug 3, 2020. It is now read-only.

hs:ccd:export filename (-f) gets ".yaml" appended #5

Open
tkn98 opened this issue Jul 15, 2015 · 2 comments
Open

hs:ccd:export filename (-f) gets ".yaml" appended #5

tkn98 opened this issue Jul 15, 2015 · 2 comments

Comments

@tkn98
Copy link

tkn98 commented Jul 15, 2015

This is counter-intuitive if you want to write to stdout.

For example the following command does not work:

$ n98-magerun.phar hs:ccd:export --include shipping/origin -fphp://stdout

because the following filename becomes created:

Failed to write: php://stdout.yaml

Automatically adding ".yaml" destroys the portability of the switch. I suggest to remove that feature for -f / --filename and add a new switch that brings it back: --prepend-filename-with-dot-yaml-extension.

@SchumacherFM
Copy link
Contributor

Good spot! May you send me a PR?

@tkn98
Copy link
Author

tkn98 commented Aug 5, 2015

Nice feedback! I'll take a look and file something.

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

No branches or pull requests

2 participants