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

Changes to turn D::P::F into Dancer2::Plugin::Feed #10

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

yanick
Copy link

@yanick yanick commented May 19, 2013

Here are the changed required to make the module work under Dancer2. In the off-chance you prefer not to have the maintenance of the Dancer2:: version of the module, I can release it too. :-)

Joy!
`/anick

@hobbestigrou
Copy link
Owner

2013/5/20 Yanick Champoux [email protected]:

Here are the changed required to make the module work under Dancer2. In the
off-chance you prefer not to have the maintenance of the Dancer2:: version
of the module, I can release it too. :-)
Thank you for your contribution. Sorry for the lag, I had little
problems. I am a new git repository and made a release with this
namespace. I will do the same for my other plugin, thank you again.
By-cons just a question why use die, there is no system of exeception
dancer2.

\0/ Hobbestigrou
site web: erakis.im

@yanick
Copy link
Author

yanick commented May 28, 2013

On 13-05-27 10:28 AM, hobbestigrou wrote:

Thank you for your contribution. Sorry for the lag, I had little
problems.

No problem. :-)

I am a new git repository and made a release with this
namespace.

Woohoo! \o/ Thanks!

By-cons just a question why use die, there is no system of exeception
dancer2.

There is one. I must say, I went for the quick and easy road first, as I
needed to get something running. There is probably much room for
improvement there. :-)

Joie,
`/anick

@hobbestigrou
Copy link
Owner

2013/5/28 Yanick Champoux [email protected]

Woohoo! \o/ Thanks!
You're welcome

There is one. I must say, I went for the quick and easy road first, as I
needed to get something running. There is probably much room for
improvement there. :-)
Ok, I'll watch and thanks again for your contribution.

\0/ Hobbestigrou
site web: erakis.im

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.

2 participants