Skip to content

Add recommendations for runner groups and labels#91

Open
konstruktoid wants to merge 3 commits into
github:mainfrom
konstruktoid:rungroup
Open

Add recommendations for runner groups and labels#91
konstruktoid wants to merge 3 commits into
github:mainfrom
konstruktoid:rungroup

Conversation

@konstruktoid
Copy link
Copy Markdown

This pull request makes several improvements to the documentation on securing GitHub Actions workflows. The most significant updates include adding new recommendations for segregating runners, enhancing repository ruleset guidance, and updating author attribution.

Enhancements to security recommendations:

  • Added a new recommendation to segregate runners by using organizational runner groups and labels to separate high-privilege from low-privilege runners, reducing the risk of unauthorized access to sensitive resources. [1] [2]
  • Expanded repository ruleset guidance by recommending the use of "Require workflows to pass before merging" to enforce organizational or enterprise-level workflow requirements prior to merging.

Documentation and metadata updates:

  • Updated the authors list in the document metadata to include Thomas Sjögren (konstruktoid).
  • Set the document draft status back to true, indicating it is not yet ready for publication.

Signed-off-by: Thomas Sjögren <konstruktoid@users.noreply.github.com>
Copilot AI review requested due to automatic review settings May 22, 2026 16:03
@konstruktoid konstruktoid requested review from a team as code owners May 22, 2026 16:03
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Updates the “Securing GitHub Actions Workflows” guidance by adding runner-segregation recommendations and expanding ruleset guidance, while marking the page as draft.

Changes:

  • Set the page to draft: true and added an additional author.
  • Added “Segregate runners” as a top-level recommendation and a dedicated section with implementation details.
  • Added a repository ruleset recommendation to require workflows to pass before merging.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +4 to 6
draft: true # Set to false when ready to publish
title: 'Securing GitHub Actions Workflows'
publishDate: 2024-08-16
Comment thread content/library/application-security/recommendations/actions-security/index.md Outdated
Comment thread content/library/application-security/recommendations/actions-security/index.md Outdated
Signed-off-by: Thomas Sjögren <konstruktoid@users.noreply.github.com>
Signed-off-by: Thomas Sjögren <konstruktoid@users.noreply.github.com>
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.

2 participants