Skip to content
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

planet API is making the CI crash #519

Closed
12rambau opened this issue Jul 3, 2022 · 3 comments
Closed

planet API is making the CI crash #519

12rambau opened this issue Jul 3, 2022 · 3 comments
Labels
bug Something isn't working cd/ci relative to CI

Comments

@12rambau
Copy link
Member

12rambau commented Jul 3, 2022

All the rececent CI based on master are crashing as new deprecation warning are raised during the Jupyter-sphinx execution:

WARNING: Cell printed to stderr:
21
/opt/hostedtoolcache/Python/3.8.13/x64/lib/python3.8/site-packages/planet/api/init.py:38: ClientV1DeprecationWarning: The planet.api module is deprecated and will be removed in version 2.0.0. For more details please see the discussion at https://github.com/planetlabs/planet-client-python/discussions.

I think we need to have a look to see if we need to fix planet version

@12rambau 12rambau added bug Something isn't working cd/ci relative to CI labels Jul 3, 2022
@12rambau
Copy link
Member Author

12rambau commented Jul 4, 2022

The solution might be here: jupyter/jupyter-sphinx#178

@dfguerrerom
Copy link
Collaborator

Anyways I think we have to move to version 2. v1 is ugly.

@12rambau
Copy link
Member Author

12rambau commented Jul 4, 2022

I made sure sphinx stop throwing warning to our face, we'll move to v2 whenever we want

@12rambau 12rambau closed this as completed Jul 4, 2022
This was referenced Aug 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working cd/ci relative to CI
Projects
None yet
Development

No branches or pull requests

2 participants