Skip to content

tested and added root org support #22

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

chandrajeetn
Copy link
Collaborator

No description provided.

@chandrajeetn chandrajeetn requested a review from Copilot May 13, 2025 23:05
Copy link

@Copilot 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 adds and tests support for retrieving and integrating root organization data using the LUMS API. Key changes include the introduction of a new rootOrg package with an HTTP client, poller, and configuration; updates to the evaluation logic in the local evaluation package; and minor import and naming adjustments.

Reviewed Changes

Copilot reviewed 9 out of 10 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
pkg/rootOrg/rootOrg.go Introduces LUMS API client for fetching root organization
pkg/rootOrg/poller.go Implements a poller for periodically triggering requests
pkg/rootOrg/config.go Adds configuration settings for root organization support
pkg/rootOrg/client.go Provides a client for root org integration and evaluation
pkg/experiment/local/client.go Minor import reordering and update in client usage
model/lums.go Defines models for LUMS API responses
localEvaluation/localEvaluation.go Updates initialization and evaluation logic to include root org support
Files not reviewed (1)
  • go.mod: Language not supported

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.

1 participant