Skip to content

[Github Actions] Added monorepo_source_of_truth flag#2211

Merged
amd-hsivasun merged 3 commits into
developfrom
users/hsivasun/repos-config-flag
Nov 5, 2025
Merged

[Github Actions] Added monorepo_source_of_truth flag#2211
amd-hsivasun merged 3 commits into
developfrom
users/hsivasun/repos-config-flag

Conversation

@amd-hsivasun
Copy link
Copy Markdown
Collaborator

Motivation

Create another flag per project on repos-config.json to present the super-repo being the source of truth, instead of the old individual repo. This allows projects to not have this source of truth designation tied to auto-subtree-push, as we may not want that behaviour for all projects on the super repo and eventually want to turn off this feature.

Technical Details

Test Plan

Test Result

Submission Checklist

Copy link
Copy Markdown
Collaborator

@geomin12 geomin12 left a comment

Choose a reason for hiding this comment

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

lgtm, can we also open a GH issue to track the eventual removal of this? or maybe add a comment somewhere about removing that flag once it becomes source of truth?

@amd-hsivasun
Copy link
Copy Markdown
Collaborator Author

lgtm, can we also open a GH issue to track the eventual removal of this? or maybe add a comment somewhere about removing that flag once it becomes source of truth?

The flag I added already indicates that the monorepo is the source of truth but I have created an issue to track the eventual disabling of the subtree-push to update the individual repos: #2226

Copy link
Copy Markdown
Contributor

@alexxu-amd alexxu-amd left a comment

Choose a reason for hiding this comment

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

LGTM

@amd-hsivasun amd-hsivasun merged commit 314df1b into develop Nov 5, 2025
19 of 20 checks passed
@amd-hsivasun amd-hsivasun deleted the users/hsivasun/repos-config-flag branch November 5, 2025 16:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants