Skip to content

Commit e9290a7

Browse files
remove smoke-test hack (#431)
1 parent 6880dce commit e9290a7

3 files changed

Lines changed: 21 additions & 9 deletions

File tree

packages/google-cloud-videointelligence/synth.metadata

Lines changed: 16 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,25 @@
11
{
2-
"updateTime": "2020-04-11T01:05:57.076030Z",
32
"sources": [
3+
{
4+
"git": {
5+
"name": ".",
6+
"remote": "git@github.com:googleapis/nodejs-video-intelligence.git",
7+
"sha": "dfb36d2b24d32e6a8de41d4ecda9900de224b4d0"
8+
}
9+
},
10+
{
11+
"git": {
12+
"name": "googleapis",
13+
"remote": "https://github.com/googleapis/googleapis.git",
14+
"sha": "42ee97c1b93a0e3759bbba3013da309f670a90ab",
15+
"internalRef": "307114445"
16+
}
17+
},
418
{
519
"git": {
620
"name": "synthtool",
721
"remote": "https://github.com/googleapis/synthtool.git",
8-
"sha": "6f32150677c9784f3c3a7e1949472bd29c9d72c5",
9-
"log": "6f32150677c9784f3c3a7e1949472bd29c9d72c5\nfix: installs test_utils from its common repo (#480)\n\n\n74ce986d3b5431eb66985e9a00c4eb45295a4020\nfix: stop recording update_time in synth.metadata (#478)\n\n\n7f8e62aa3edd225f76347a16f92e400661fdfb52\nchore(java): release-please only updates non maven versions in README (#476)\n\nPrevent release-please and synthtool from fighting over the released library version. Synthtool updates the install snippets from the samples pom.xml files so the bots fight if they are temporarily out of sync after a release.\nc7e0e517d7f46f77bebd27da2e5afcaa6eee7e25\nbuild(java): fix nightly integration test config to run integrations (#465)\n\nThis was only running the units.\nbd69a2aa7b70875f3c988e269706b22fefbef40e\nbuild(java): fix retry_with_backoff when -e option set (#475)\n\n\nd9b173c427bfa0c6cca818233562e7e8841a357c\nfix: record version of working repo in synth.metadata (#473)\n\nPartial revert of b37cf74d12e9a42b9de9e61a4f26133d7cd9c168.\nf73a541770d95a609e5be6bf6b3b220d17cefcbe\nfeat(discogapic): allow local discovery-artifact-manager (#474)\n\n\n8cf0f5d93a70c3dcb0b4999d3152c46d4d9264bf\ndoc: describe the Autosynth & Synthtool protocol (#472)\n\n* doc: describe the Autosynth & Synthtool protocol\n\n* Accommodate review comments.\n980baaa738a1ad8fa02b4fdbd56be075ee77ece5\nfix: pin sphinx to <3.0.0 as new version causes new error (#471)\n\nThe error `toctree contains reference to document changlelog that doesn't have a title: no link will be generated` occurs as of 3.0.0. Pinning to 2.x until we address the docs build issue.\n\nTowards #470\n\nI did this manually for python-datastore https://github.com/googleapis/python-datastore/pull/22\n928b2998ac5023e7c7e254ab935f9ef022455aad\nchore(deps): update dependency com.google.cloud.samples:shared-configuration to v1.0.15 (#466)\n\nCo-authored-by: Jeffrey Rennie <rennie@google.com>\n188f1b1d53181f739b98f8aa5d40cfe99eb90c47\nfix: allow local and external deps to be specified (#469)\n\nModify noxfile.py to allow local and external dependencies for\nsystem tests to be specified.\n1df68ed6735ddce6797d0f83641a731c3c3f75b4\nfix: apache license URL (#468)\n\n\nf4a59efa54808c4b958263de87bc666ce41e415f\nfeat: Add discogapic support for GAPICBazel generation (#459)\n\n* feat: Add discogapic support for GAPICBazel generation\n\n* reformat with black\n\n* Rename source repository variable\n\nCo-authored-by: Jeffrey Rennie <rennie@google.com>\n99820243d348191bc9c634f2b48ddf65096285ed\nfix: update template files for Node.js libraries (#463)\n\n\n3cbe6bcd5623139ac9834c43818424ddca5430cb\nfix(ruby): remove dead troubleshooting link from generated auth guide (#462)\n\n\na003d8655d3ebec2bbbd5fc3898e91e152265c67\ndocs: remove \"install stable\" instructions (#461)\n\nThe package hasn't been released to PyPI in some time\nf5e8c88d9870d8aa4eb43fa0b39f07e02bfbe4df\nchore(python): add license headers to config files; make small tweaks to templates (#458)\n\n\ne36822bfa0acb355502dab391b8ef9c4f30208d8\nchore(java): treat samples shared configuration dependency update as chore (#457)\n\n\n1b4cc80a7aaf164f6241937dd87f3bd1f4149e0c\nfix: do not run node 8 CI (#456)\n\n\nee4330a0e5f4b93978e8683fbda8e6d4148326b7\nchore(java_templates): mark version bumps of current library as a chore (#452)\n\nWith the samples/install-without-bom/pom.xml referencing the latest released library, we want to mark updates of this version as a chore for renovate bot.\na0d3133a5d45544a66345059eebf76933265c099\nfix(java): run mvn install with retry (#453)\n\n* fix(java): run mvn install with retry\n\n* fix invocation of command\n"
22+
"sha": "19465d3ec5e5acdb01521d8f3bddd311bcbee28d"
1023
}
1124
}
1225
],

packages/google-cloud-videointelligence/synth.py

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -18,15 +18,14 @@
1818
version,
1919
generator_args={
2020
"grpc-service-config": f"google/cloud/videointelligence/{version}/videointelligence_grpc_service_config.json",
21-
"package-name": f"@google-cloud/video-intelligence",
22-
"main-service": f"videointelligence",
21+
"package-name": "@google-cloud/video-intelligence",
22+
"main-service": "videointelligence",
2323
},
2424
proto_path=f'/google/cloud/videointelligence/{version}',
2525
)
2626

2727
# skip index, protos, package.json, and README.md
28-
s.copy(library, excludes=["package.json", "README.md", "src/index.ts",
29-
"smoke-test/video_intelligence_service_smoke_test.ts"])
28+
s.copy(library, excludes=["package.json", "README.md", "src/index.ts"])
3029

3130
templates = common_templates.node_library(source_location='build/src')
3231
s.copy(templates)

packages/google-cloud-videointelligence/system-test/install.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ import {readFileSync} from 'fs';
2121
import {describe, it} from 'mocha';
2222

2323
describe('typescript consumer tests', () => {
24-
it('should have correct type signature for typescript users', async function() {
24+
it('should have correct type signature for typescript users', async function () {
2525
this.timeout(300000);
2626
const options = {
2727
packageDir: process.cwd(), // path to your module.
@@ -35,7 +35,7 @@ describe('typescript consumer tests', () => {
3535
await packNTest(options); // will throw upon error.
3636
});
3737

38-
it('should have correct type signature for javascript users', async function() {
38+
it('should have correct type signature for javascript users', async function () {
3939
this.timeout(300000);
4040
const options = {
4141
packageDir: process.cwd(), // path to your module.

0 commit comments

Comments
 (0)