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

Prevent invalid warning when var ENV 'ORB_PARAM_ORB_PUB_TOKEN' isn't define #167

Merged

Conversation

benoittgt
Copy link
Contributor

I know the resolution of this issue can appears weird. But it just happen to me and I had to look at the source code to understand.
Also I think the doc should mention that it is required.

This is ORB_PARAM_ORB_PUB_TOKEN but it is defined via a parameter with a default as CIRCLE_TOKEN.

If you set ORB_PARAM_ORB_PUB_TOKEN in a context it will not work with the orb template but if you set CIRCLE_TOKEN it is working properly.

This is ORB_PARAM_ORB_PUB_TOKEN but it is defined via a parameter with a
default as CIRCLE_TOKEN.

If you set ORB_PARAM_ORB_PUB_TOKEN in a context it will not work with
the orb template but if you set CIRCLE_TOKEN it is working properly.
@benoittgt benoittgt changed the title Prevent invalid warning when var ENV isn't define Prevent invalid warning when var ENV 'ORB_PARAM_ORB_PUB_TOKEN' isn't define Oct 5, 2022
@KyleTryon
Copy link
Contributor

Thank you @benoittgt 🙏 . I see what happened here, ORB_PARAM_ORB_PUB_TOKEN is actually somewhat correct, but only to the internally scoped step, so even if you set this, it would not work. So This is perfect 👍

@KyleTryon KyleTryon merged commit b2058d6 into CircleCI-Public:master Dec 12, 2022
@orb-publisher
Copy link

Your development orb has been published. It will expire in 30 days.
You can preview what this will look like on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/circleci/orb-tools?version=dev:b2058d635bed47809091dfa22ba76d19d1cb5a69

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.

None yet

3 participants