Skip to content
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

Get closure-library from google-closure-library. #2509

Merged
merged 3 commits into from
Jan 9, 2020

Conversation

mikelehen
Copy link
Contributor

Get closure-library from google-closure-library instead of via closure-builder so we can control the version directly.

…e-builder so we can control the version directly.
Copy link
Contributor

@schmidt-sebastian schmidt-sebastian left a comment

Choose a reason for hiding this comment

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

As a Closure expert, this LGTM to me.

(I did check and it builds fine for me).

@@ -16,6 +16,7 @@
"license": "Apache-2.0",
"devDependencies": {
"closure-builder": "2.3.8",
"google-closure-library": "^20200101.0.0",
Copy link
Contributor

Choose a reason for hiding this comment

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

We don't use version ranges in the Web SDK. Not that it matters for this versioning scheme, but it might be better to be consistent and drop the ^.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oops! I just did "npm install --save-dev" or some such... Thanks. I'll remember this for next time!

@mikelehen
Copy link
Contributor Author

@hsubox76 Can you approve the yarn.lock change?

Copy link
Contributor

@hsubox76 hsubox76 left a comment

Choose a reason for hiding this comment

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

Looks good!

@mikelehen mikelehen merged commit e9bf41c into master Jan 9, 2020
@mikelehen mikelehen deleted the mikelehen/use-closure-library-npm-dependency branch January 9, 2020 20:57
@hsubox76 hsubox76 added this to the next milestone Jan 9, 2020
@firebase firebase locked and limited conversation to collaborators Feb 9, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants