Skip to content
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

chore(dev): move all component repo-metadata.json files to repo-metadata-full.json #7281

Merged
merged 3 commits into from
May 3, 2024

Conversation

bshaffer
Copy link
Contributor

@bshaffer bshaffer commented May 2, 2024

fixes #7280

Since we will be bumping each component's release, we might as well make this change so we don't have to re-release everything again.

@bshaffer bshaffer requested review from a team as code owners May 2, 2024 18:18
@bshaffer bshaffer added the next release PRs to be included in the next release label May 2, 2024
@bshaffer
Copy link
Contributor Author

bshaffer commented May 2, 2024

@chingor13 could you just give this a quick LGTM to make sure it looks like the right syntax?

@@ -0,0 +1,1445 @@
{
"google/cloud-access-approval": {
Copy link
Contributor

Choose a reason for hiding this comment

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

This should be the path to the .repo-metadata.json would have been

Copy link
Contributor

@chingor13 chingor13 left a comment

Choose a reason for hiding this comment

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

LGTM with the assumption that the JSON fully parses (e.g. no extra trailing commas)

@bshaffer bshaffer enabled auto-merge (squash) May 3, 2024 16:29
@bshaffer bshaffer merged commit 0a5e5f6 into main May 3, 2024
30 checks passed
@bshaffer bshaffer deleted the consolidate-repo-metadata branch May 3, 2024 16:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
next release PRs to be included in the next release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feature Request: move all .repo-metadata.json into one .repo-metadata-full.json
3 participants