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

Abstract out Configuration Parsing #390

Open
balupton opened this issue Jan 2, 2013 · 4 comments
Open

Abstract out Configuration Parsing #390

balupton opened this issue Jan 2, 2013 · 4 comments

Comments

@balupton
Copy link
Member

balupton commented Jan 2, 2013

We could probably abstract out our configuration parsing.


Want to back this issue? Place a bounty on it! We accept bounties via Bountysource.

@ghost ghost assigned balupton Jan 2, 2013
@przemoli
Copy link

Yes!

Split config into multiple files.

Having helper functions for eco alongside of build config seam strange..

Simpliest thing to do would be to keep current syntax but with import keyword allowing to include content of any "valid" configuration file.

That way developer can keep their specific configs in separate files.

@balupton
Copy link
Member Author

@przemoli thanks! btw, you can require and merge multiple configuration files now already by using docpad.coffee as your configuration file then someData = require('some-data.coffee') to include data from another configuration file. Example here and here

@Zearin
Copy link
Contributor

Zearin commented Sep 26, 2013

👍

(Also, thanks for the tip, @balupton ! Gonna start using that right away!)

@Zearin
Copy link
Contributor

Zearin commented Apr 19, 2015

Was looking over the source of lib/docpad.coffee.

Holy hell, that is one monstrously long god-class!!!

A huge part of it is config-related. Definitely worth marking this as a priority item.

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

No branches or pull requests

3 participants