-
Notifications
You must be signed in to change notification settings - Fork 4
Update tests #10
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
Update tests #10
Conversation
|
@seignovert: it seems we need to update the |
|
I updated the pylint configuration and very slightly edited some files to be compliant with flake8, pylint and bandit. |
|
I also updated the expected results in the example notebooks and the docs. |
|
All checks are now passing fine. @seignovert : I let you do the merge when you prefer. |
|
Hello @BaptisteCecconi, Everything looks good to me 👍 Thank you very much for this update. |
Since I'm willing to add new functionalities on the package, I wanted to make sure that the tests were running correctly before starting.
I realised some tests were failing because remote data changes at ESA and NASA.
Here is a PR that fixes those issues.