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

Adds support for SoyToIncrDomSrcCompiler from https://github.com/goog… #225

Merged
merged 1 commit into from
Apr 21, 2017

Conversation

jbalsas
Copy link
Contributor

@jbalsas jbalsas commented Apr 4, 2017

…le/closure-templates/commit/7dae5a1a2b6868da939e7643741b67db1610f190

@jbalsas
Copy link
Contributor Author

jbalsas commented Apr 4, 2017

I thought this was ready, but I'm seeing a test fail with the new compiler, so I'll try to see if it needs something else!

@jbalsas jbalsas closed this Apr 4, 2017
@jbalsas
Copy link
Contributor Author

jbalsas commented Apr 12, 2017

Hey @eduardolundgren, this is actually good to merge!

The problem is that in this project we're still using [email protected], so when I update, I hit this change: metal/metal-tools-soy@9e5163d

Whenever we want to update the compiler version, we just need to also change https://github.com/metal/metal.js/blob/master/gulpfile.js#L100 and we're good to go!

@jbalsas jbalsas reopened this Apr 12, 2017
@jbalsas jbalsas requested a review from eduardolundgren April 12, 2017 14:45
@eduardolundgren eduardolundgren merged commit 1430daa into master Apr 21, 2017
@jbalsas jbalsas deleted the incrdom_compiler branch October 10, 2017 10:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants