Skip to content

Conversation

@ssherar
Copy link
Collaborator

@ssherar ssherar commented Sep 3, 2025

No description provided.

@ssherar ssherar requested a review from Copilot September 3, 2025 10:01
Copy link

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

This PR extends the GitHub plugin to collect teams information in addition to organization settings. It restructures the data model to wrap both organization settings and teams data in a unified GithubData struct.

  • Introduces a new GithubData struct to hold both organization settings and teams information
  • Updates data fetching to retrieve teams data from the GitHub API
  • Modifies policy evaluation to work with the new data structure

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
main.go Added empty line for formatting
internal/data.go Added GithubData struct and updated FetchData to retrieve teams information
internal/eval.go Updated policy evaluator to work with new GithubData struct instead of raw organization
internal/eval_test.go Updated test data and function calls to use new data structure
examples/policies/gh_org_mfa_enabled.rego Updated policy to reference input.settings instead of input.organization
README.md Updated documentation formatting and added teams permission requirement
Makefile Added new Makefile with policy testing and validation targets

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@ssherar ssherar merged commit f930635 into main Sep 3, 2025
1 check passed
@ssherar ssherar deleted the feat/teams branch September 3, 2025 10:13
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