Skip to content
This repository was archived by the owner on Apr 3, 2024. It is now read-only.

chore: license check as posttest#384

Merged
jinwoo merged 2 commits intogoogleapis:masterfrom
jinwoo:jsgl
Jan 8, 2018
Merged

chore: license check as posttest#384
jinwoo merged 2 commits intogoogleapis:masterfrom
jinwoo:jsgl

Conversation

@jinwoo
Copy link
Copy Markdown
Contributor

@jinwoo jinwoo commented Jan 5, 2018

There are two problematic dependencies: log-driver & modelo. They come
from googleapis/nodejs-common. I created an issue for that:
googleapis/nodejs-common#16. Until it's
resolved, this repo must have a package whitelist.

There are two problematic dependencies: log-driver & modelo. They come
from googleapis/nodejs-common. I created an issue for that:
googleapis/nodejs-common#16. Until it's
resolved, this repo must have a package whitelist.
@jinwoo jinwoo requested review from a team, DominicKramer and ofrobots January 5, 2018 18:36
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jan 5, 2018
Copy link
Copy Markdown
Contributor

@DominicKramer DominicKramer left a comment

Choose a reason for hiding this comment

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

Can you also add an issue to this repo to remove log-driver from the packageWhitelist when @google-cloud/common has been updated?

@@ -0,0 +1,5 @@
{
"packageWhitelist": [
"log-driver"

This comment was marked as spam.

This comment was marked as spam.

@jinwoo
Copy link
Copy Markdown
Contributor Author

jinwoo commented Jan 8, 2018

Filed #387

@jinwoo jinwoo merged commit 3b2a3ef into googleapis:master Jan 8, 2018
@jinwoo jinwoo deleted the jsgl branch January 8, 2018 19:50
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