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

[push] Suggest other command if dp.json is missing #29

Closed
wants to merge 1 commit into from

Conversation

AcckiyGerman
Copy link
Contributor

Suggest two other commands if dp.json is missing.
datopian/datahub-qa#63

@AcckiyGerman
Copy link
Contributor Author

@Mikanebu could you please review & merge this PR. Here is the original issue:
datopian/datahub-qa#63

Copy link
Member

@anuveyatsu anuveyatsu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@AcckiyGerman I don't think CLI related information should be in this library - we're trying to make it independent so other projects can use it not only data-cli. So either this fix should be somewhere in the data-cli or datahub-client (not sure if it is relevant).

@AcckiyGerman
Copy link
Contributor Author

Ok will catch error and add help message in data-cli part

@AcckiyGerman AcckiyGerman deleted the fix/data_push_error_msg branch February 13, 2018 08:30
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

2 participants