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

many core plugins do not make use of dump_config #266

Closed
karenetheridge opened this issue Jan 14, 2014 · 1 comment
Closed

many core plugins do not make use of dump_config #266

karenetheridge opened this issue Jan 14, 2014 · 1 comment

Comments

@karenetheridge
Copy link
Contributor

(placeholder, to be replaced with a patch eventually.. if no one else beats me to it -- note to others, this is a nice entry-level piece of work that does not require understanding huge amounts of dzil internals)

Many plugins in the core Dist::Zilla distribution do not make good use of dump_config() to include all their configuration settings in metadata (included in META.* by the [MetaConfig] plugin). Debugging the output of a dist is easier with this, so they should be added.

(Also possibly think about how to extract configs from roles applied to plugins, such as Pod::Elemental::PerlMunger which is applied to [PodWeaver] -- yes I know PodWeaver isn't core -- it would be nice to get the 'replacer' config and friends in the dumped config too, even though the base class itself doesn't know anything about them. Could use some sort of method modifier?)

@rjbs
Copy link
Owner

rjbs commented Feb 8, 2014

This is now #276

@rjbs rjbs closed this as completed Feb 8, 2014
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

No branches or pull requests

2 participants