eddiewebb / circleci-queue

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

fix(urlencode): urlencode CIRCLE_BRANCH [semver:patch] #90

Closed andrew-barnett closed 1 year ago

andrew-barnett commented 1 year ago

Motivation, issues

The CircleCI API (started?) requiring urlencoded branch names, otherwise the calls were returning empty results. This caused the queue to never succeed. This fix urlencodes the branch name. See #89

Description

add urlencode function and run ${CIRCLE_BRANCH} through it while building branch-specific API url

fixes #89

eddiewebb commented 1 year ago

bors try

eddiewebb commented 1 year ago

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

bors[bot] commented 1 year ago

try

Build succeeded:

eddiewebb commented 1 year ago

bors delegate+

bors[bot] commented 1 year ago

:v: andrew-barnett can now approve this pull request. To approve and merge a pull request, simply reply with bors r+. More detailed instructions are available here.

eddiewebb commented 1 year ago

@andrew-barnett - please see my comment/concern about portability. Assuming it works across those 3 shells I have delegated review permission, so you can just add a comment here of bors merge

andrew-barnett commented 1 year ago

@eddiewebb I'm not finding your comment about which shells to check. Can you post here?

eddiewebb commented 1 year ago

I'm sorry Andrew it was stuck pending, submitted it.

andrew-barnett commented 1 year ago

@andrew-barnett - please see my comment/concern about portability. Assuming it works across those 3 shells I have delegated review permission, so you can just add a comment here of bors merge

@eddiewebb - It didn't work on zsh, so updated the code to make it portable. Passing it back for review.

andrew-barnett commented 1 year ago

bors try

eddiewebb commented 1 year ago

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

bors[bot] commented 1 year ago

try

Build succeeded:

andrew-barnett commented 1 year ago

bors r+

eddiewebb commented 1 year ago

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

bors[bot] commented 1 year ago

Build failed:

eddiewebb commented 1 year ago

bors merge

bors[bot] commented 1 year ago

Merge conflict.

eddiewebb commented 1 year ago

( ialways get the magic PR title wrong, added a sanity check and changelog)

Version 1.8.2 and soon 1.8.3 wil include this fix, thanks @andrew-barnett !

eddiewebb commented 1 year ago

bors merge

eddiewebb commented 1 year ago

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

eddiewebb commented 1 year ago

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

bors[bot] commented 1 year ago

Build succeeded: