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

Update readthedocs script #76

Merged
merged 3 commits into from
Apr 19, 2024
Merged

Update readthedocs script #76

merged 3 commits into from
Apr 19, 2024

Conversation

luisaforozco
Copy link
Collaborator

Context

Docs were failing due to a change in readthedocs handling of virtual environments with poetry: see issues 11149 and 11150, as well as PR solving them.

With these changes, readthedocs documentation, and specifically instruction to use poetry backend was updated.

Changes

In this PR, I updated .readthedocs.yaml accordingly to the new instructions.

Tests

I've tested the deployment of this branch docs_pip in readthedocs: succesfull build | docs

@luisaforozco luisaforozco self-assigned this Apr 19, 2024
Copy link

sonarcloud bot commented Apr 19, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@chyalexcheng chyalexcheng merged commit c2d1d31 into main Apr 19, 2024
10 of 15 checks passed
@luisaforozco luisaforozco deleted the docs_pip branch April 19, 2024 19:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants