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.

Sign up for GitHub

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

Jump to bottom

Module::Starter should support modules with more than one author. [rt.cpan.org #7627] #25

Open
xsawyerx opened this issue Jan 5, 2015 · 0 comments
Open

Module::Starter should support modules with more than one author. [rt.cpan.org #7627] #25

xsawyerx opened this issue Jan 5, 2015 · 0 comments
Labels
Migrated From RT Wishlist

Comments

@xsawyerx
Copy link
Owner

xsawyerx commented Jan 5, 2015

https://rt.cpan.org/Ticket/Display.html?id=7627

In today's world of collaborative software development, it makes sense that the next generation of Perl module creation utilities should facilitate a group of developers working on a single module or set of modules. Module::Store currently lacks this functionality.

Thanks in advance for any work towards implementing this.
mcmillhj pushed a commit to mcmillhj/module-starter that referenced this issue Nov 29, 2015
Add support for multiple authors to Module::Starter; Issue xsawyerx#25
5a17f4a 
The included changes add support for multiple authors to Module::Starter and propogate to the chosen builder option. To do this, it seemed appropriate to remove the --email option and ask that users' specify both the author name and email as a single option, which most of the builders expect anyway:

    module-starter --module=Foo::Bar,Foo::Bat \
        --author="Andy Lester <[email protected]> \
        --author="Sawyer X <[email protected]>

The attribute that previously held the author, `$self->{author}` now holds an arrayref of authoremails. Which are passed in turn to Module::Build and ExtUtils::MakeMaker as arrayrefs and to Module::Install as a string.

I tried to follow the existing conventions that were used within the module, but please let me know if I missed anything and/or there are other change you would like me to make.
@mcmillhj mcmillhj mentioned this issue Nov 29, 2015
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Migrated From RT Wishlist
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@xsawyerx