Skip to content
This repository was archived by the owner on Mar 17, 2026. It is now read-only.

feat: add schema revision samples#1870

Merged
gcf-merge-on-green[bot] merged 33 commits into
googleapis:mainfrom
feywind:schema-revisions
Jan 31, 2024
Merged

feat: add schema revision samples#1870
gcf-merge-on-green[bot] merged 33 commits into
googleapis:mainfrom
feywind:schema-revisions

Conversation

@feywind
Copy link
Copy Markdown
Collaborator

@feywind feywind commented Jan 5, 2024

This adds support for easily getting a schema gapic client, along with samples for schema revisions.

@feywind feywind requested review from a team January 5, 2024 19:23
@snippet-bot
Copy link
Copy Markdown

snippet-bot Bot commented Jan 5, 2024

Here is the summary of changes.

You are about to add 18 region tags.

This comment is generated by snippet-bot.
If you find problems with this result, please file an issue at:
https://github.com/googleapis/repo-automation-bots/issues.
To update this comment, add snippet-bot:force-run label or use the checkbox below:

  • Refresh this comment

@product-auto-label product-auto-label Bot added the size: l Pull request size is large. label Jan 5, 2024
@product-auto-label product-auto-label Bot added api: pubsub Issues related to the googleapis/nodejs-pubsub API. samples Issues that are directly related to samples. labels Jan 5, 2024
@feywind feywind changed the title feat: add schema revision wrappers and new samples feat: add schema revision samples Jan 8, 2024
@feywind feywind added the owlbot:run Add this label to trigger the Owlbot post processor. label Jan 8, 2024
@gcf-owl-bot gcf-owl-bot Bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Jan 8, 2024
@generated-files-bot
Copy link
Copy Markdown

Warning: This pull request is touching the following templated files:

@product-auto-label product-auto-label Bot added size: xl Pull request size is extra large. and removed size: l Pull request size is large. labels Jan 10, 2024
@feywind feywind added the owlbot:run Add this label to trigger the Owlbot post processor. label Jan 10, 2024
@gcf-owl-bot gcf-owl-bot Bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Jan 10, 2024
@feywind feywind added the owlbot:run Add this label to trigger the Owlbot post processor. label Jan 12, 2024
@gcf-owl-bot gcf-owl-bot Bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Jan 12, 2024
Copy link
Copy Markdown
Member

@hongalex hongalex left a comment

Choose a reason for hiding this comment

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

These samples look good. I think there are a few missing ones,most important is probably pubsub_get_schema_revision but also pubsub_commit_proto_schema and pubsub_subscribe_avro_records_with_revisions. The first two should probably go in this PR, while the latter two can be in a separate PR?

Comment thread samples/README.md Outdated
Comment thread samples/deleteSchemaRevision.js Outdated
Comment thread samples/typescript/deleteSchemaRevision.ts Outdated
@feywind feywind added the owlbot:run Add this label to trigger the Owlbot post processor. label Jan 22, 2024
@gcf-owl-bot gcf-owl-bot Bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Jan 22, 2024
@feywind
Copy link
Copy Markdown
Collaborator Author

feywind commented Jan 22, 2024

These samples look good. I think there are a few missing ones,most important is probably pubsub_get_schema_revision but also pubsub_commit_proto_schema and pubsub_subscribe_avro_records_with_revisions. The first two should probably go in this PR, while the latter two can be in a separate PR?

Hmm, those didn't make it into my hotlist. I'll just add them in this PR.

@feywind feywind added the owlbot:run Add this label to trigger the Owlbot post processor. label Jan 31, 2024
@gcf-owl-bot gcf-owl-bot Bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Jan 31, 2024
@feywind feywind added the automerge Merge the pull request once unit tests and other checks pass. label Jan 31, 2024
@feywind feywind added the owlbot:run Add this label to trigger the Owlbot post processor. label Jan 31, 2024
@gcf-owl-bot gcf-owl-bot Bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Jan 31, 2024
@gcf-merge-on-green gcf-merge-on-green Bot merged commit 044e149 into googleapis:main Jan 31, 2024
@gcf-merge-on-green gcf-merge-on-green Bot removed the automerge Merge the pull request once unit tests and other checks pass. label Jan 31, 2024
@feywind feywind deleted the schema-revisions branch January 31, 2024 19:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: pubsub Issues related to the googleapis/nodejs-pubsub API. samples Issues that are directly related to samples. size: xl Pull request size is extra large.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants