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
DM-13869: Implement basic Registry.find and related functionality #23
Conversation
a6c5730
to
965952f
Compare
This really is just a bare-bones starting point. Does no lookup based on unit values and thus can only handle one dataset per collection. This is of course utterly useless except for building on.
965952f
to
3857fbe
Compare
I'd be wary of running autopep8 on the entire code base when we know we are already flake8 clean. |
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 is fine. I'd like us to stop running autopep8 in the future given that we already pass flake8. DM-13869 says it is meant to be working on Butler.get but I don't think this is (and I'm working on that).
Running autopep8 is my lazy way of being flake8 compatible (at least for the multi-line sql annoyance). But I'll stop running it on a non-file-to-file basis now. |
Can you enable flake8 checking in your editor? It's just I'm worried about autopep8 fiddling with things that currently are fine. |
No description provided.