-
Notifications
You must be signed in to change notification settings - Fork 110
Update default values in everest config schema #10431
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
base: main
Are you sure you want to change the base?
Conversation
CodSpeed Performance ReportMerging #10431 will not alter performanceComparing Summary
|
6696e83
to
70d2379
Compare
@@ -11,7 +11,6 @@ def test_input_constraint_initialization(): | |||
cfg = relpath(cfg_dir, "config_input_constraints.yml") | |||
config = EverestConfig.load_file(cfg) | |||
# Check that an input constraint has been defined |
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.
I think this comment can be removed now
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.
Magical 🧙♂️
Some tests are still failing
70d2379
to
c74dc60
Compare
9fedf0b
to
b252f80
Compare
b252f80
to
48870b6
Compare
This will remove a lot of the "invalid" default values from the json schema and improve the documentation.
Old:

New:

Approach
Short description of the approach
(Screenshot of new behavior in GUI if applicable)
git rebase -i main --exec 'just rapid-tests'
)When applicable