Skip to content

Comments

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

Closed
stdclass wants to merge 2 commits intogithub-tools:masterfrom
stdclass:master
Closed

Added support for node.js / require()#4
stdclass wants to merge 2 commits intogithub-tools:masterfrom
stdclass:master

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