bzarras / newsapi

A promise-based node interface for NewsAPI

MIT License

144 stars 39 forks source link

Fix #18 and upgrade Packages to latest version #23

Closed yololftw closed 4 years ago

yololftw commented 4 years ago

18 I was facing the same issue

I upgraded all the packages to the latest version (I just started Node and this was made by trial and error, So please guide me if I did things wrong!) I ran the test and it worked fine :+1: When I ran build I was getting no output only a warning that said 'npm WARN' build 'npm build' called with no arguments. Did you mean to 'npm run-script build?' I am not sure what is needed here Also I did not upgrade the version please feel free to bump it

bzarras commented 4 years ago

Just updated packages myself. Going to close this one, but I appreciate you bringing this to my attention!

yololftw commented 4 years ago

No Problem! Thanks for updating the package