-
Notifications
You must be signed in to change notification settings - Fork 21
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
Check in synth.py and conform to google node repo standards #14
Conversation
@GoogleCloudPlatform/node-team - need help enabling Circle for this one :) |
Need to add the docs directory to the .gitignore :) |
@kinwa91 do you have all the rights you need to modify the CircleCI env vars and stuff? |
Yup. I will put in the good stuff. :) |
Do tests in this repo call the actual service? @stephenplusplus |
Yes, they do. There aren't many traditional unit tests-- the testing was mainly done using the actual service. |
Got it. I'll start a thread to move those to system-test and rewrite some unit tests. |
fixes #11