BytewaveMLP / node-derpi

A (WIP) Derpibooru API wrapper for Node.js
MIT License
7 stars 2 forks source link

Remove duplicate assignment of default request options #11

Closed nullforce closed 4 years ago

nullforce commented 4 years ago

Closes #10

BytewaveMLP commented 4 years ago

Thanks for the PR! Appreciate the cleanup and test fixes.

At some point I should get around to cleaning up this project as a whole... especially with the new API changes going on at Derpi.

I noticed this is still marked as a WIP; did you still have changes you wanted to make, or was that by mistake?

nullforce commented 4 years ago

Should be marked as ready now.

BytewaveMLP commented 4 years ago

Thanks, just wanted to make sure this was ready to merge.

Thanks again for the contribution!