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

samples: add schema samples, typescript sample support #1262

Merged
merged 59 commits into from
Jun 7, 2021

Conversation

feywind
Copy link
Collaborator

@feywind feywind commented Apr 16, 2021

fixes: #1220

The remainder of this PR is about adding schema sample snippets, as well as some preliminary TypeScript snippet support. There are still some linting issues with the generated samples, and I wanted to talk about the method used here before committing to it, so it's still marked as a work in progress.

@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 Apr 16, 2021
@snippet-bot
Copy link

snippet-bot bot commented Apr 16, 2021

Here is the summary of changes.

You are about to add 20 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

@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Apr 16, 2021
samples/createTopicWithSchema.js Outdated Show resolved Hide resolved
samples/createTopicWithSchema.js Outdated Show resolved Hide resolved
…ma-support

# Conflicts:
#	src/pubsub.ts
#	test/pubsub.ts
@feywind feywind changed the title feat: schema support, typescript based sample tests samples: add schema samples, typescript sample support May 21, 2021
@anguillanneuf anguillanneuf marked this pull request as ready for review June 3, 2021 17:30
@anguillanneuf anguillanneuf requested review from a team as code owners June 3, 2021 17:30
@anguillanneuf anguillanneuf self-requested a review June 3, 2021 17:30
Copy link
Contributor

@anguillanneuf anguillanneuf left a comment

Choose a reason for hiding this comment

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

See inline comments.

samples/listenForAvroRecords.js Outdated Show resolved Hide resolved
samples/listenForProtobufMessages.js Outdated Show resolved Hide resolved
samples/publishAvroRecords.js Outdated Show resolved Hide resolved
samples/publishProtobufMessages.js Outdated Show resolved Hide resolved
samples/typescript/listenForAvroRecords.ts Outdated Show resolved Hide resolved
samples/typescript/listenForProtobufMessages.ts Outdated Show resolved Hide resolved
samples/typescript/publishAvroRecords.ts Outdated Show resolved Hide resolved
samples/typescript/publishProtobufMessages.ts Outdated Show resolved Hide resolved
@anguillanneuf anguillanneuf self-requested a review June 4, 2021 19:29
Copy link
Contributor

@anguillanneuf anguillanneuf left a comment

Choose a reason for hiding this comment

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

LGTM to me overall. I'm not a Node.js expert. 😃

@feywind feywind added the automerge Merge the pull request once unit tests and other checks pass. label Jun 7, 2021
@gcf-merge-on-green gcf-merge-on-green bot merged commit f9dc4c6 into googleapis:master Jun 7, 2021
@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Jun 7, 2021
feywind pushed a commit to feywind/nodejs-pubsub that referenced this pull request Nov 12, 2024
Source-Link: googleapis/synthtool@3602660
Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-nodejs:latest@sha256:e6d785d6de3cab027f6213d95ccedab4cab3811b0d3172b78db2216faa182e32

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: pubsub Issues related to the googleapis/nodejs-pubsub API. cla: yes This human has signed the Contributor License Agreement. samples Issues that are directly related to samples.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add support for Pub/Sub server schema to the Node client library
2 participants