search

duckduckgo / p5-app-duckpan

DuckDuckHack OpenSource Development Application
http://metacpan.org/module/App::DuckPAN
Other
54 stars 47 forks source link

Add InstantAnswer::Config #344

Closed GuiltyDolphin closed 8 years ago

GuiltyDolphin commented 8 years ago

This provides a wrapper around the metadata from the platform; allowing us to provide additional information and operations that aren't present in the metadata.

Depends on #343.

/cc @zachthompson

zachthompson commented 8 years ago

What operations have enough use to warrant looking into this?

GuiltyDolphin commented 8 years ago

@zachthompson Yeah - sorry I haven't made this super clear.

I'm breaking up #335 into smaller pieces so we can pick and choose what we want.

A great use case for this in particular is when detecting files - instead of using templates to find which directories to monitor, we can grab each InstantAnswer::Config's files and monitor those. The main user of this is Restarter.

I'm still figuring out what goes with what, and the dependencies - so would you like me to outline all the changes in #335 and then you can say which features you think are acceptable?

If you think having Restarter use the files from InstantAnswer::Config is a good use case I can pull those changes into this PR.

GuiltyDolphin commented 8 years ago

@zachthompson This would be very useful for commands like duckpan rm, duckpan mv etc. as we contain the 'meta' information along with files, making it easy to find which files need to change for the given command.

At the least I think we need to pull out the file-finding based on metadata (as in #343) so that we have a way of finding files independent of templates.