-
-
Notifications
You must be signed in to change notification settings - Fork 7.4k
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
proposal: Optional externalize site config #3090
Comments
I like this generic and flexible approach - it's optional can be done as you like. Putting thinks like author profiles, menu definitions etc. in different data files can definitely improve the maintenance of a large config file. But how do you distinguish between |
The last one does not make sense the way we parse data files. It will be an object with key "language.en". But in general, we have a "last key will win" approach (with warning or error logging on duplicates). I have not analyzed every corner of this proposal, but I think it should be doable. |
Also note that all of the site config that we, for some reason, put in external data files, will have to logically follow the same life cycle as the site config (think live reload), so making it "one thing" makes sense. |
This issue has been automatically marked as stale because it has not had recent activity. The resources of the Hugo team are limited, and so we are asking for your help. |
This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
This relates to #3088 -- re. @digitalcraftsman comment:
So
author
configuration is site configuration, but because it may potentially be lots of them, we should put them in some special place.I suggest rather that we make this a general thing:
Site config belongs in
config.toml
and cousins, but it can optionally be put in files below/data/_config/
. Keys inconfig.toml
will always win.@digitalcraftsman this one is open for discussion.
The text was updated successfully, but these errors were encountered: