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

Fix curl code and HTTP status code checking so deploy will fail when … #14

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

ww-mgr
Copy link

@ww-mgr ww-mgr commented Dec 28, 2017

Fix curl code and HTTP status code checking so deploy will fail when credentials aren't correct.

HTTP status code and process exit code checking seems to be wrong. Please review code changes.

@doncorsean
Copy link

doncorsean commented Jan 11, 2019

Can we get this merged? Kinda important to fail if artifact is preexisting in a releases repo. @ww-mgr

Also @ww-mgr. Awesome last name, how many times a day do you tell people "I am Groot?"

@dagda1
Copy link

dagda1 commented Oct 10, 2019

Hi, Is there any chance of getting this merged?

@gigaga
Copy link

gigaga commented Jan 28, 2020

I have the same problem...
Please @cthorne66 can you merge this PR and publish v0.1.9?
That will be great!

@gigaga
Copy link

gigaga commented Nov 5, 2020

Up!

@gigaga
Copy link

gigaga commented Aug 24, 2022

For my part, I forked it and replaced curl command by request node module...

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.

5 participants