CircleCI-Public / orb-tools-orb

Various tools for authoring and publishing CircleCI orbs
https://circleci.com/orbs/registry/orb/circleci/orb-tools
MIT License
51 stars 74 forks source link

Checkout parameter of publish job is not used. #211

Open eddiewebb opened 1 year ago

eddiewebb commented 1 year ago

Orb version:

12.0.3

What happened:

While supporting client and looking at means to inject custom logic I noticed the publish job will optinoally attach workspace and then publish, but unlike review/pack jobs never runs checkout, regardless of parameter (step does not exist)

Expected behavior:

Remove parameter or use it in a conditional checkout step.

Additional Information:

kelvintaywl commented 1 year ago

the checkout (bool) parameter is part of the publish job indeed: https://github.com/CircleCI-Public/orb-tools-orb/blob/7b7c2918f9bcc4663539288eedb0a1fa8ce125dd/src/jobs/publish.yml#L78-L82

however, there is no actual conditional step to checkout based on this parameter (i.e., the parameter is not used)

KyleTryon commented 1 year ago

Thanks folks, ill take a look at this shortly.

jtatarik commented 1 year ago

This issue is still present in 12.0.4. Have to work around it by adding explicit checkout pre-step.