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

Update error handling for creating new product #32

Merged

Conversation

lubomir
Copy link
Contributor

@lubomir lubomir commented Dec 2, 2019

Start the process by cloning the configuration repo. This should catch cases where the default system-wide configuration does not work at all. An error message is printed in this case telling the user where to put proper configuration.

It is still possible the clone will work, the product will be created and then pushing will fail (for example because the user lacks permission). In this case the content to add to the configuration file is still printed.

Start the process by cloning the configuration repo. This should catch
cases where the default system-wide configuration does not work at all.
An error message is printed in this case telling the user where to put
proper configuration.

It is still possible the clone will work, the product will be created
and then pushing will fail (for example because the user lacks
permission). In this case the content to add to the configuration file
is still printed.
@lubomir lubomir merged commit cf45c7a into release-engineering:master Dec 3, 2019
@lubomir lubomir deleted the better-errors-on-new-product branch December 3, 2019 09:59
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

1 participant