-
Notifications
You must be signed in to change notification settings - Fork 61k
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
Actions: Add section about group/ungroup commands #482
Conversation
Hm, should I retake the screenshot at 150% or 200% browser scale? That would make it look crisper on high density screens. |
Thanks so much @Simran-B! I'll get this triaged for review 🎉 |
The screen shot looks much better on my phone now, but it's displayed quite large on desktop. Is there an existing solution to adjust the size based on native pixel size or something along this line? |
This PR is stale because it has been open 7 days with no activity and will be automatically closed in 3 days. To keep this PR open, update the PR by adding a comment or pushing a commit. |
Sorry about the delay @Simran-B! We've got quite a few things going on that our small team is taking care of, but this is definitely not stale and we'll get it reviewed 💛 |
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.
@Simran-B thanks for catching this and updating the docs! ✨ I just had one suggestion to change the language to match the tone style guide for the docs. Once that is updated, I can get this shipped for you! 🚢
content/actions/reference/workflow-commands-for-github-actions.md
Outdated
Show resolved
Hide resolved
Co-authored-by: Simran <Simran-B@users.noreply.github.com>
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.
@Simran-B thank you so much for cleaning up the table and adding the group command! This looks fantastic. 🙌 I'm going to ship this right away. 🚀 🚢
Thanks very much for contributing! Your pull request has been merged 🎉 You should see your changes appear on the site in approximately 24 hours. |
Why:
Missing in Actions documentation, but available here:
https://github.com/actions/toolkit/blob/main/docs/commands.md#group-and-ungroup-log-lines
What's being changed:
Changed headline level:
Check off the following:
All of the tests are passing.in staginglocally.