-
Notifications
You must be signed in to change notification settings - Fork 71
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
✨ Add needs_from_toml
configuration
#1337
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #1337 +/- ##
==========================================
+ Coverage 86.87% 88.00% +1.12%
==========================================
Files 56 60 +4
Lines 6532 7070 +538
==========================================
+ Hits 5675 6222 +547
+ Misses 857 848 -9
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
This reverts commit 71b8d8d.
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.
Minor wording questions
This configuration allows for a path to a toml file to be specified.
In this case, needs configuration will be read from it and merged into the config from the
conf.py
.An additional
needs_from_toml_table
config controls the toml table to read from.