Skip to content
This repository has been archived by the owner on Feb 26, 2024. It is now read-only.

fix: commonjs wrapper #22

Closed
wants to merge 1 commit into from
Closed

fix: commonjs wrapper #22

wants to merge 1 commit into from

Conversation

btford
Copy link
Contributor

@btford btford commented Jun 4, 2014

Closes #19

@btford
Copy link
Contributor Author

btford commented Jun 4, 2014

@pedroteixeira can you try this branch and see if it resolves your issue? Thanks!

@btford btford added cla: yes and removed cla: no labels Jun 4, 2014
@darzu
Copy link

darzu commented Jul 21, 2014

I pull your branch and gave it a try (I'm running into issue #19 as well) and no longer get the error in patchSetClearFn but now I get the same error in Zone.patchableFn (line 244).

I'll try to write a test for this.

Update: writing a test was foiled by #28

@kentcdodds
Copy link
Member

Would love for this to get merged! Trying to setup using Angular 2 with webpack. Having this fixed would simplify life significantly :-)

@btford btford closed this Apr 10, 2015
@mhevery mhevery deleted the fix-common-js branch December 19, 2016 18:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Does not seem to be browserify compatible
3 participants