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

Make more Node-y #2

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

alexellis
Copy link

@alexellis alexellis commented Jan 19, 2017

What I did

  • Read should be asynchronous
  • Eliminate toString() by specifying the encoding as UTF8
  • Use up to date ES2015 style via "use strict"

Why I did it

  • You stared my minio-gauge repo 👍
  • Liked your blog entry (reminded me of a post I wrote on alexellis.io)
  • Node.js commercial experience

Btw

Would be great if you could try out my coding challenge over at: https://github.com/alexellis/xservedbyfinder

Looking for implementations that solve the problem in various programming languages. It's not a hard problem and you will see plenty of examples.

* Read should be asynchronous
* Eliminate toString() by specifying the encoding as UTF8
* Use up to date ES2015 style via "use strict"
@alexellis
Copy link
Author

.. ping ..

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.

1 participant