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: [publicca] new module for publicca #8557

Merged
merged 2 commits into from
Oct 10, 2022

Conversation

suztomo
Copy link
Member

@suztomo suztomo commented Oct 7, 2022

  • feat: initial generation of publicca
  • build: add the publicca module to monorepo

@suztomo suztomo added the owlbot:run Add this label to trigger the Owlbot post processor. label Oct 7, 2022
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Oct 7, 2022
Comment on lines 16 to 19
deep-remove-regex:
- "/grpc-google-.*/src"
- "/proto-google-.*/src"
- "/google-.*/src"
Copy link
Contributor

Choose a reason for hiding this comment

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

Shouldn't this prefixed with /java-publicca?

Copy link
Member Author

Choose a reason for hiding this comment

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

Good catch.

@suztomo suztomo closed this Oct 7, 2022
@suztomo suztomo reopened this Oct 7, 2022
Comment on lines +16 to +22
deep-remove-regex:
- "/java-publicca/grpc-google-.*/src"
- "/java-publicca/proto-google-.*/src"
- "/java-publicca/google-.*/src"

deep-preserve-regex:
- "/java-publicca/google-.*/src/test/java/com/google/cloud/.*/v.*/it/IT.*Test.java"
Copy link
Member Author

Choose a reason for hiding this comment

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

@lqiu96 Fixed. Now they have "/java-publicca" at the beginning.

@@ -384,6 +384,10 @@
"component": "google-cloud-profiler",
"skip-github-release": true
},
"java-publicca": {
"component": "google-cloud-publicca",
"skip-github-release": true
Copy link
Member

Choose a reason for hiding this comment

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

What does skip-github-release do?

Copy link
Member Author

Choose a reason for hiding this comment

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

Not to create a GitHub release entry for now during the shadow mode.

@suztomo suztomo added automerge Merge the pull request once unit tests and other checks pass. owlbot:run Add this label to trigger the Owlbot post processor. labels Oct 10, 2022
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Oct 10, 2022
@suztomo suztomo merged commit f32e69b into googleapis:main Oct 10, 2022
@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Oct 10, 2022
@suztomo suztomo deleted the new_module_java-publicca branch October 10, 2022 13:30
@release-please release-please bot mentioned this pull request Nov 10, 2022
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.

None yet

3 participants