Skip to content

Important fix #6

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

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Important fix #6

wants to merge 6 commits into from

Conversation

akidee
Copy link

@akidee akidee commented May 10, 2011

No description provided.

@danieldickison
Copy link

See pull request 6 for another way to achieve ab63de6 without breaking Module/1.1 compatibility.

@akidee
Copy link
Author

akidee commented Aug 24, 2011

What I am looking for is a simple wrapper so that I can use client side modules in the same way I use node.js modules.
What can you do about https://github.com/akidee/yabble/commit/043a88ea2e45f4daf97defc90ca0081ee5e6adbc ?

@danieldickison
Copy link

I'm confused by 043a88e -- it looks like you're trying to enabled something like require('http://server.com/some-module'), but I don't think node.js lets you do this. I can see it being more useful in a browser, however.

(My patch does not address this -- it just addresses the module.exports = function ()... idiom.)

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.

None yet

2 participants