docs icon indicating copy to clipboard operation
docs copied to clipboard

Add additional shell script examples for the new Job output environment variables

Open SimonDarksideJ opened this issue 2 years ago • 5 comments

Why:

The documentation about the new Output variables for sharing data between jobs was inaccurate as it only handled the default Ubuntu "Bash" version of the commands.

What's being changed (if available, include any code snippets, screenshots, or gifs):

This updates Job output variables to denote the different versions used for each shell language, kept original Bash, added Powershell and Python

Check off the following:

  • [ ] I have reviewed my changes in staging (look for the "Automatically generated comment" and click the links in the "Preview" column to view your latest changes).
  • [X] For content changes, I have completed the self-review checklist.

SimonDarksideJ avatar Nov 24 '22 19:11 SimonDarksideJ

Thanks for opening this pull request! A GitHub docs team member should be by to give feedback soon. In the meantime, please check out the contributing guidelines.

welcome[bot] avatar Nov 24 '22 19:11 welcome[bot]

@SimonDarksideJ Thanks so much for opening a PR! I'll get this triaged for review :zap:

cmwilson21 avatar Nov 25 '22 17:11 cmwilson21

All in the day as an MVP :S

SimonDarksideJ avatar Nov 25 '22 17:11 SimonDarksideJ

apeche

mrt-erl avatar Dec 02 '22 18:12 mrt-erl

Why.

CERVANTESJERARDO48 avatar Dec 14 '22 00:12 CERVANTESJERARDO48

@SimonDarksideJ I'll be closing this PR due to inactivity - but if you would like to work on this further, we're happy to reopen this PR 💛

nguyenalex836 avatar Jan 11 '24 20:01 nguyenalex836

To be honest @nguyenalex836 I had given up on this PR as it did not seem desired, even though it was to address an issue I came across in the docs.

SimonDarksideJ avatar Jan 11 '24 20:01 SimonDarksideJ

I'm sorry to hear that, @SimonDarksideJ! Let us know if you'd like to reopen this PR - we're happy to do so 💛

nguyenalex836 avatar Jan 11 '24 22:01 nguyenalex836