-
Notifications
You must be signed in to change notification settings - Fork 60.8k
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
Remove shell injection in example #22826
Conversation
@soliton- Thanks so much for opening a PR! I'll get this triaged for review ⚡ |
@cmwilson21 Just a reminder. This probably got lost over the holidays. Is there anything I need to do? |
@soliton- Thanks for checking in! It's up on the board waiting for review. Someone will have eyes on it soon. 👀 We do appreciate your patience as we are catching up after the holidays 💛 |
@soliton- Thanks again for your patience here! I checked with the team and this looks like a great addition to the docs. I'll get this merged down once all the tests have passed ✅ Thank you for your interest in improving GitHub Docs! 💖 |
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.
LGTM 🚢
Thanks very much for contributing! Your pull request has been merged 🎉 You should see your changes appear on the site in approximately 24 hours. If you're looking for your next contribution, check out our help wanted issues ⚡ |
Why:
Avoid shell injection in workflow example.
What's being changed (if available, include any code snippets, screenshots, or gifs):
Workflow example
Check off the following: