Skip to content

Make LoaderMap a singleton instance#37

Open
rictic wants to merge 1 commit into
masterfrom
loadermap-class
Open

Make LoaderMap a singleton instance#37
rictic wants to merge 1 commit into
masterfrom
loadermap-class

Conversation

@rictic

@rictic rictic commented Aug 28, 2018

Copy link
Copy Markdown
Contributor

This lets closure compiler understand it much better, allowing us to turn on --collapse_properties, which greatly reduces binary size

This lets closure compiler understand it much better, allowing us to turn on --collapse_properties, which greatly reduces binary size
@rictic
rictic requested a review from e111077 August 28, 2018 16:25
@rictic

rictic commented Aug 28, 2018

Copy link
Copy Markdown
Contributor Author

All internal tests pass.

@aomarks

aomarks commented Aug 28, 2018

Copy link
Copy Markdown
Contributor

Let's wait for the final 3.0 releases before merging this, just to avoid unnecessary churn.

cc @bicknellr

@rictic

rictic commented Aug 28, 2018

Copy link
Copy Markdown
Contributor Author

Any reason for that? We've been merging other no-op internal changes

@e111077

e111077 commented Aug 28, 2018

Copy link
Copy Markdown
Contributor

Wouldn't you want to get this in before the next release so that it's available in all the upcoming tests with that release?

@bicknellr

Copy link
Copy Markdown
Contributor

Please don't merge this until the final release.

@bicknellr bicknellr closed this Sep 14, 2018
@bicknellr
bicknellr changed the base branch from __auto_generated_3.0_preview to master September 14, 2018 22:45
@bicknellr bicknellr reopened this Sep 14, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants