- Notifications
You must be signed in to change notification settings - Fork673
Improve helper example section with env var usage#1580
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
a8e7060 tob9722c8CompareAdded an example for using envrionement variable for tokens.
| private_token = helper: printf -- '%%s' ${GITLAB_TOKEN} | ||
| timeout = 1 | ||
| **Notice:** | ||
| * ``printf`` is the executable provided by coreutils package (on a debian based distribution). | ||
| * ``%`` must be escaped using an other `%` (in order to allow `configparser` load configuration properly. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Thank you for this contribution@glennie! Initially I wanted to say this would probably be superseded by#1296, but I like your idea because you can define multiple env vars (e.g. one per gitlab id entry).
Just one question: Wouldn't simply usingecho -n ${VARIABLE} achieve the same? Have you tried this?-n should strip the newline.
This Pull Request (PR) was marked stale because it has been open 90 days with no activity. Please remove the stale label or comment on this PR. Otherwise, it will be closed in 15 days. |
This PR was closed because it has been marked stale for 15 days with no activity. If this PR is still valid, please re-open. |
Hi,
I wanted to use an env var for the token.
I spend some time to figure out that I needed to escape '%' because of configparser.
Therefore, I made this PR to improve the current documentation hoping others will spend much less time than me.
Kind regards,