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

Regenerate library with synth.py customizations#345

Merged
JustinBeckwith merged 13 commits intomasterfrom
run-synth
Sep 11, 2018
Merged

Regenerate library with synth.py customizations#345
JustinBeckwith merged 13 commits intomasterfrom
run-synth

Conversation

@jkwlui
Copy link
Copy Markdown
Contributor

@jkwlui jkwlui commented Sep 11, 2018

firestore had a number of unique pattern that warrants a customized synth.py:

  • code are in /dev
  • init library like so: const client = require('@google-cloud/firestore').v1()
  • .jsdoc.js needs to pull source from build/
  • uses its own project (ok since cannot share a project with one using datastore) - running synthtool from HEAD (nodejs: add test_project arg for templates synthtool#63)

system-test and samples-test can be verified in this branch (run-synth-system-tests):
https://circleci.com/gh/googleapis/workflows/nodejs-firestore/tree/run-synth-system-tests
(in-sync with this branch except enabling system/samples-test in .circleci configs)

@ghost ghost assigned jkwlui Sep 11, 2018
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Sep 11, 2018
Comment thread .circleci/config.yml
Comment thread .nycrc
"build/src/*/v*/*.js",
"build/protos/*.js",
"build/test/**/*.js"
"src/*{/*,/**/*}.js",

This comment was marked as spam.

This comment was marked as spam.

@JustinBeckwith
Copy link
Copy Markdown
Contributor

Also, thank you for doing this @kinwa91

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants