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
Rewrite script #87
Rewrite script #87
Conversation
dev-scripts/prepare_pecl_release.py
Outdated
except subprocess.CalledProcessError as e: | ||
print(f'ERROR PROCESSING review package.xml: {e}') | ||
exit(1) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd rather we NOT try and catch the exception, so it's clear which line caused the error.
the cause/effect will be more clear if we print out the cmd being run (see above suggestion about run() helper)
or put the try/catch in the run() helper, and you can just sys.exit('FAILED')
because who needs a stack trace, it's obvious which command failed, it's the one being run
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sure, just put a try/catch for prepare_pecl_package_xml.py
for now to remind people check package.xml
for error details.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
whatever and shipit
51c7239
to
d143ec4
Compare
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.