Skip to content
This repository was archived by the owner on Jul 20, 2023. It is now read-only.

feat: add code samples and tests for overlay creation - #39

Merged
bcoe merged 3 commits into
googleapis:masterfrom
irataxy:master
Mar 12, 2021
Merged

bcoe merged 3 commits into
googleapis:masterfrom
irataxy:master

Conversation

@irataxy

@irataxy irataxy commented Mar 10, 2021

Copy link
Copy Markdown
Contributor

b:181782257

@irataxy
irataxy requested a review from a team March 10, 2021 18:35
@product-auto-label product-auto-label Bot added the api: transcoder Issues related to the googleapis/nodejs-video-transcoder API. label Mar 10, 2021
@snippet-bot

snippet-bot Bot commented Mar 10, 2021 •

Copy link
Copy Markdown

Here is the summary of changes.

You are about to add 2 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 samples Issues that are directly related to samples. label Mar 10, 2021
@google-cla google-cla Bot added the cla: yes This human has signed the Contributor License Agreement. label Mar 10, 2021
@codecov

codecov Bot commented Mar 10, 2021 •

Copy link
Copy Markdown

Codecov Report

Merging #39 (9d65c25) into master (ecac4fd) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #39   +/-   ##
=======================================
  Coverage   96.20%   96.20%           
=======================================
  Files           5        5           
  Lines        1581     1581           
  Branches       61       61           
=======================================
  Hits         1521     1521           
  Misses         59       59           
  Partials        1        1           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ecac4fd...9d65c25. Read the comment docs.

@bcoe bcoe left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Just a small nit, we usually hyphen-case JavaScript files, so:

create-job-with-static-overlay.js

Comment thread package.json Outdated
Comment thread samples/createJobWithAnimatedOverlay.js
Comment thread samples/test/transcoder.test.js
@@ -0,0 +1,119 @@
/**
* Copyright 2021, Google, Inc.
* Licensed under the Apache License, Version 2.0 (the "License");

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

nit: we usually hyphen case our Node.js samples.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Ack. Do I need to change this? I will then have to rename every .js file in the samples.

@bcoe
bcoe merged commit 47686fa into googleapis:master Mar 12, 2021
gcf-merge-on-green Bot pushed a commit that referenced this pull request Mar 15, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: transcoder Issues related to the googleapis/nodejs-video-transcoder 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.

2 participants