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

Get rid of stdin for deis config:push #305

Open
pfeodrippe opened this issue May 23, 2017 · 2 comments
Open

Get rid of stdin for deis config:push #305

pfeodrippe opened this issue May 23, 2017 · 2 comments
Labels

Comments

@pfeodrippe
Copy link

pfeodrippe commented May 23, 2017

The implementation of deis config:push here uses stdin to fetch the buffer, but it was causing me problems as you can see at this issue. Running the command inside a script which gets its inputs from stdin, I had to pipe it with < dev/null to get the correct result or it would give me Creating config... ...Error: Unknown Error (409): {"detail":"user changed nothing - release stopped"}.

Can something be made to correct it?

@pfeodrippe pfeodrippe changed the title Get rid of stdin for deis config:push Get rid of stdin for deis config:push May 23, 2017
@Joshua-Anderson
Copy link
Contributor

Looks like we need to switch code to check if a filename is provided before attempting to use stdin.

@Cryptophobia
Copy link

This issue was moved to teamhephy/workflow-cli#6

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

No branches or pull requests

3 participants