Skip to content

Create a fresh copy of the base repository config #795

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

Merged

Conversation

klutchell
Copy link
Contributor

@klutchell klutchell commented Mar 25, 2025

Avoid duplicating base repository settings when initializing repo config.

@Copilot Copilot AI review requested due to automatic review settings March 25, 2025 16:46
@klutchell klutchell force-pushed the kyle/new-base-repo-object branch from b6edc69 to 287522a Compare March 25, 2025 16:47
Copy link
Contributor

@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 updates the repository configuration logic to create a fresh copy of the base repository config instead of referencing the original object.

  • Uses Object.assign to shallow copy the base repository config
  • Merges repository-specific properties into the fresh copy without mutating the original

@klutchell klutchell force-pushed the kyle/new-base-repo-object branch from 287522a to 352fddf Compare March 25, 2025 16:48
Avoid duplicating base repository settings when initializing repo config.

Signed-off-by: Kyle Harding <[email protected]>
@klutchell klutchell force-pushed the kyle/new-base-repo-object branch from 352fddf to b082cd5 Compare March 25, 2025 16:58
@klutchell
Copy link
Contributor Author

@decyjphr this one fixed a bug for us where all repos were inheriting the repo name setting from a discovered repo called .sentry

Copy link
Collaborator

@decyjphr decyjphr left a comment

Choose a reason for hiding this comment

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

👍🏾

@decyjphr decyjphr merged commit 28b3dfd into github:main-enterprise Mar 25, 2025
2 checks passed
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