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

feat(parametermanager): created folder to add samples #2921

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

arpangoswami
Copy link

feat(parametermanager): created folder to add samples

Added a placeholder file for folder creation of parametermanager

Added cloud-parameters-team to blunderbuss.yml and CODEOWNERS

@arpangoswami arpangoswami requested a review from a team as a code owner February 25, 2025 08:57
@product-auto-label product-auto-label bot added the samples Issues that are directly related to samples. label Feb 25, 2025
Copy link
Member

@amanda-tarafa amanda-tarafa left a comment

Choose a reason for hiding this comment

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

Also, when you start writing the actual samples, make sure to

  • Follow the contributing guide
  • Several smaller PRs are better than a huge one. It's easier to review and bring to completion one small PR than a bigger one. Learnings from a small PR can be more easily applied to subsequent PRs.

@@ -45,6 +45,10 @@ assign_issues_by:
- 'api: secretmanager'
to:
- GoogleCloudPlatform/cloud-secrets-team
- labels:
- 'api: parametermanager'
Copy link
Member

Choose a reason for hiding this comment

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

This label does not exist, they are synced to GitHub from an internal list, you need to add it internally firts and then wait for it to sync back here.

- labels:
- 'api: parametermanager'
to:
- GoogleCloudPlatform/cloud-parameters-team
Copy link
Member

Choose a reason for hiding this comment

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

This group does not exist.

@@ -30,6 +30,7 @@ appengine/flexible/ @GoogleCloudPlatform/serverless-runtimes @GoogleCloudPlatfor

kms/ @GoogleCloudPlatform/dotnet-samples-reviewers
secretmanager/ @GoogleCloudPlatform/dotnet-samples-reviewers @GoogleCloudPlatform/cloud-secrets-team
secretmanager/ @GoogleCloudPlatform/dotnet-samples-reviewers @GoogleCloudPlatform/cloud-secrets-team @GoogleCloudPlatform/cloud-parameters-team
Copy link
Member

Choose a reason for hiding this comment

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

You copy pasted but didn't make all changes, this is still setting ownership for secretmanager.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
samples Issues that are directly related to samples.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants