-
Notifications
You must be signed in to change notification settings - Fork 43
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
Improve error handling #689
Improve error handling #689
Conversation
This package has no library so doctests don't work Move the doctest into regular tests
I can't figure out which problem is solved by c7fed77 "Fix the detection of unknown attributes from config set/add" |
OK, this now works:
|
In some cases, the old code implemented 1-line methods that just returned |
I would recommend generating the documentation for |
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.
LGTM
agama config set
#688.flatten
(in serde jargon) fields.agama_settings
documentation to explain better how it works.Click for a documentation screenshot 😉