-
Notifications
You must be signed in to change notification settings - Fork 7
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 Model Permissions Tests #80
base: main
Are you sure you want to change the base?
Conversation
Hello @dannygoldstein! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:
|
The template app is supposed to be the one users clone to start out from with baselayer. Thoughts? These tests are good to add, so I'm tempted to merge this. We need to sort out the CI problems though. |
CI errors are unrelated to current PR:
|
This PR adds tests for cesium-ml/baselayer#184 that will be run by the baselayer CI.