Closed veganism closed 8 years ago
I forked to rework the endpoints, but then it occurred to me that you may prefer to rename the repo as well. @mlbright, would you rather I wait until you've done that before I submit a PR?
@JeffMelton done!
@mlbright Thanks! Question, though: It looks like you already have a module in CPAN named DarkSky::API, based on an even earlier version of their API. I'm happy to work on renaming files, updating endpoints and documentation. Would this new DarkSky::API replace that one altogether, or would you prefer to leave that one and call this something different?
Let's replace it. Out with the old, in with the new.
On Fri, Sep 23, 2016 at 9:20 AM, Jeff Melton notifications@github.com wrote:
@mlbright https://github.com/mlbright Thanks! Question, though: It looks like you already have a module in CPAN named DarkSky::API, based on an even earlier version of their API. I'm happy to work on renaming files, updating endpoints and documentation. Would this new DarkSky::API replace that one altogether, or would you prefer to leave that one and call this something different?
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/mlbright/DarkSky-API/issues/3#issuecomment-249190617, or mute the thread https://github.com/notifications/unsubscribe-auth/AAAkvlXIUcjod4uc2zux-uddNYQ1tRyeks5qs9IGgaJpZM4KBwQe .
Perfect. You've got a PR from me built on that pre-emptive assumption. :)
On Fri, Sep 23, 2016 at 01:22:39PM CDT, Martin-Louis Bright wrote:
Let's replace it. Out with the old, in with the new.
On Fri, Sep 23, 2016 at 9:20 AM, Jeff Melton notifications@github.com wrote:
@mlbright https://github.com/mlbright Thanks! Question, though: It looks like you already have a module in CPAN named DarkSky::API, based on an even earlier version of their API. I'm happy to work on renaming files, updating endpoints and documentation. Would this new DarkSky::API replace that one altogether, or would you prefer to leave that one and call this something different?
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/mlbright/DarkSky-API/issues/3#issuecomment-249190617, or mute the thread https://github.com/notifications/unsubscribe-auth/AAAkvlXIUcjod4uc2zux-uddNYQ1tRyeks5qs9IGgaJpZM4KBwQe .
You are receiving this because you were mentioned. Reply to this email directly or view it on GitHub: https://github.com/mlbright/DarkSky-API/issues/3#issuecomment-249266476
Link to blog post
Everything needs renamed apparently.