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

Hi MFontany #2

Closed
wants to merge 2 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@mishin
Contributor

mishin commented Mar 15, 2015

1) I add 
# ABSTRACT: Dancer::Plugin::MemcachedFast - Cache things using Cache::Memcached::Fast
Because MetaCPAN 
doesn’t display an abstract for the module, for example.
 If you search for “MemcachedFast” in MetaCPAN your
 module doesn’t have an abstract after the title line, unlike the others
2) add
=head1 DESCRIPTION

This plugin allows Dancer to use L<Cache::Memcached::Fast> to get and store content
on a number of memcached servers.

because 
 also the documentation doesn’t have the usual structure of SYNOPSIS followed by DESCRIPTION

it's according to  #pr-challenge 
@mfontani

This comment has been minimized.

Owner

mfontani commented Mar 15, 2015

Merged the non-controversial bits (the original PR had a few files in .build which really didn't need to be there, a tidied version of a few source files, etc). Closing.

@mfontani mfontani closed this Mar 15, 2015

@mishin

This comment has been minimized.

Contributor

mishin commented Mar 15, 2015

yes, I agree, but mainly ,that you correct distribution ;))

@mishin

This comment has been minimized.

Contributor

mishin commented Mar 15, 2015

thanks for CONTRIBUTORS ;))

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment