-
Notifications
You must be signed in to change notification settings - Fork 14
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
feat(melpo+config): Initial "platform configuration" support, starting with Melpo #202
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
some minor suggestions, LMKWYT
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
overall, this looks good!
one question is that it seems like there isn't currently a way to declare defaults when the config is declared, so the config file has to specify everything. is this desirable, or should we allow leaving a field blank to let it be defaulted?
This PR is the initial work towards implementing #199.
This introduces a
config
crate, which provides tools for platforms to have compile time defined configuration as a toml file.This PR only implements these changes for
melpomene
, as adding support to hardware platforms may require more invasive crate re-orgs.