search

blakmatrix / node-migrator-bot

Migrate your old Node.js Repos
MIT License
31 stars 3 forks source link

Make it check for path.exists / path.existsSync #19

Closed glennblock closed 12 years ago

glennblock commented 12 years ago

With recent migrating of APIs from path to FS this would be useful.

blakmatrix commented 12 years ago

Yes very good idea Glenn, In my next iteration I will do path.existsSync --> fs.existsSync and maybe also tty.setRawMode --> process.stdin.setRawMode() , but first I'm going implement your suggestion of having multiple "tasks" and also as a result I figure it would be good to allow for making additional commits per task.

glennblock commented 12 years ago

Awesome.

On Thu, Jul 5, 2012 at 2:44 AM, Farrin Reid < reply@reply.github.com

wrote:

Yes very good idea glenn, In my next iteration I will do path.existsSync --> fs.existsSync and maybe also tty.setRawMode --> process.stdin.setRawMode() , but first I'm going implement your suggestion of having multiple "tasks" and also as a result I figure it would be good to allow for making additional commits per task.

Reply to this email directly or view it on GitHub:

https://github.com/blakmatrix/node-migrator-bot/issues/19#issuecomment-6765790

blakmatrix commented 12 years ago

blakmatrix/node-migrator-bot#f7d7a3786cbd5b3cf9426fd3e791d3237f19866d