Skip to content

Added support for node.js / require() #4

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

Closed
wants to merge 2 commits into from

Conversation

stdclass
Copy link
Contributor

No description provided.

@michael
Copy link
Collaborator

michael commented May 10, 2012

Hi thanks for the addition. However I'm pretty sure this doesn't work with Node yet, since it depends on jQuery's $.ajax(). ;) If you could find a decent strategy to either use $.ajax or Nodes ClientRequest I could safely merge your changes in. Even better would be getting rid of the jQuery dependency.

@michael
Copy link
Collaborator

michael commented Jul 2, 2012

I consider switching to superagent for requests. Then Github.js can be used from Node as well.

@dongyuwei
Copy link

why not just use the raw XMLHttpRequest?

then in nodejs you can:
npm install -g xmlhttprequest
https://github.com/driverdan/node-XMLHttpRequest

@al6x
Copy link

al6x commented Aug 24, 2012

Maybe it would be possible to use Browserify? It has node.js http module implemented for browser https://github.com/substack/http-browserify

@max-mapper
Copy link
Collaborator

this works with browserify now

@max-mapper max-mapper closed this Jul 8, 2013
punchagan pushed a commit to punchagan/github that referenced this pull request Aug 23, 2014
Add ability to read raw file/directory contents
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.

5 participants