-
Notifications
You must be signed in to change notification settings - Fork 2
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
Create an API route to get the config history #560
Conversation
e68ecd2
to
f7493e0
Compare
f7493e0
to
fdb6575
Compare
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!! Awesome work! Have you considered refactoring the code in load_last_from_config_history
to leverage the new method?
3d3e552
to
0dc6f98
Compare
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.
Sorry I forgot to "submit review", my comment was pending.
tests/test_modules/test_model_contracts/test_model_contract_module.py
Outdated
Show resolved
Hide resolved
68a9b73
to
74ff41f
Compare
along with a hash of each config to differentiate them easily.
so we are not tempted to call them on the objects (for example with autocomplete in PyCharm)
74ff41f
to
a1d2e9b
Compare
Description:
Create an API route to get the config history along with a hash of each config to differentiate them easily.
Checklist:
You should check all boxes before the PR is ready. If a box does not apply, check it to acknowledge it.
ran
pre-commit run --all-files
at the end.our users.
README
files and our wiki for any big design decisions, if relevant.