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

Hi! I fixed some calls to "sys" for you! #17

Merged
merged 1 commit into from Jun 23, 2012
Merged

Hi! I fixed some calls to "sys" for you! #17

merged 1 commit into from Jun 23, 2012

Conversation

node-migrator-bot
Copy link

Hello christkv!

I am migrationBot, an open-source bot, and I'm here to help you migrate your codebase to node v0.8!

Did you know that the "sys" module throws an error if your program tries to require it in node v0.8? To help keep your code running, I automatically replaced var sys = require('sys') with var util = require('util').

If you'd like to know more about these changes in node.js, take a look at nodejs/node-v0.x-archive@1582cf#L1R51 and https://github.com/joyent/node/blob/1582cfebd6719b2d2373547994b3dca5c8c569c0/ChangeLog#L51 and http://blog.jit.su/introducing-blakmatrix-and-migratorbot .

As for myself, I was written by your friendly neighborhood node ninjas at Nodejitsu, and you can find them at #nodejitsu on irc.freenode.net or with http://webchat.jit.su .

Enjoy!

--migrationBot

@jankeromnes
Copy link
Collaborator

That's a good first step to undust this library! Thanks a lot for the effort. @christkv I hope you don't mind if I merge this.

jankeromnes added a commit that referenced this pull request Jun 23, 2012
Hi! I fixed some calls to "sys" for you!
@jankeromnes jankeromnes merged commit 91f4c59 into christkv:master Jun 23, 2012
@christkv
Copy link
Owner

Nope go ahead

Sent from my iPad

On Jun 23, 2012, at 10:41 PM, Jan Keromnesreply@reply.github.com wrote:

That's a good first step to undust this library! Thanks a lot for the effort. @christkv I hope you don't mind if I merge this.


Reply to this email directly or view it on GitHub:
#17 (comment)

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

4 participants