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

General UMD support #16

Merged
merged 1 commit into from
Dec 1, 2014
Merged

General UMD support #16

merged 1 commit into from
Dec 1, 2014

Conversation

teppeis
Copy link
Contributor

@teppeis teppeis commented Nov 30, 2014

135630e broke UMD pattern, mentioned in #15 (comment) by @twada.
To resolve it, this pul-req introduces general UMD support to assert.js and test-assert.js.

@@ -256,7 +259,6 @@ var args = (function() { return arguments; })();
a.throws(makeBlock(a.deepEqual, [], args));
a.throws(makeBlock(a.deepEqual, args, []));

console.log('All OK');
Copy link
Contributor Author

Choose a reason for hiding this comment

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

move to bottom line

@teppeis teppeis mentioned this pull request Nov 30, 2014
twada added a commit to power-assert-js/espower that referenced this pull request Dec 1, 2014
@twada
Copy link
Collaborator

twada commented Dec 1, 2014

Worked for me. LGTM 👍
Thanks @teppeis!

Jxck added a commit that referenced this pull request Dec 1, 2014
@Jxck Jxck merged commit e6d3298 into Jxck:master Dec 1, 2014
@Jxck
Copy link
Owner

Jxck commented Dec 1, 2014

thanks !

@teppeis teppeis deleted the umd branch December 1, 2014 03:47
@teppeis
Copy link
Contributor Author

teppeis commented Dec 1, 2014

😸

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.

3 participants