awestendorf / node-darksky

Dark Sky API for Node.js
MIT License
26 stars 4 forks source link

export module was borked. #2

Closed saintlukas closed 11 years ago

saintlukas commented 11 years ago

the module export was in index.js was showing: module.exports = require('./lib/client').Client; this means that if I do darksky.Client(f00) I'm actually doing something like darksky.Client.Client.

This change fixed that. As a side note, I had to modify the test file (not included, because it has a key in it, and wasn't in the git pull I did, only in the NPM package for some reason.) Use the latest version of the ReadMe from the previous contributor to get the err, data syntax up to date in the callback function of the test.

This is my first git pull request ever (I've worked in Git with internal projects a bunch, though), and my first node.js work, so feel free to be thorough and let me know if I'm doing anything wrong. Thanks!