eddiewebb / circleci-queue

CircleCI orb to block/queue jobs to enforce max concurrency limits
MIT License
74 stars 75 forks source link

Remove fail flag from curl [semver:minor] #84

Closed AlexMeuer closed 2 years ago

AlexMeuer commented 2 years ago

The response code is checked anyway, so using the fail flag actually results in less helpful error messages.

Checklist

Closes #85 thanks to header inclusion from #86 , thanks @kevinr-electric and @AlexMeuer

Motivation, issues

We are seeing failures from the API when using this, but cannot debug since the -f flag for curl doesn't tell use the response code in the event of a failure.

Description

remove -f flag from curl statement

eddiewebb commented 2 years ago

bors merge

(this will publish changes from #86 as well)

bors[bot] commented 2 years ago

Merge conflict.

eddiewebb commented 2 years ago

bors merge

eddiewebb commented 2 years ago

BotComment: Development version of orb available for manual validation - eddiewebb/queue@dev:411

eddiewebb commented 2 years ago

BotComment: Production version of orb available for use - eddiewebb/queue@1.8.0

bors[bot] commented 2 years ago

Build succeeded: