Skip to content

[IMP] conf.py, requirements.txt, *: allow building with Python 3.10 #2362

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

Closed
wants to merge 2 commits into from

Conversation

Feyensv
Copy link
Collaborator

@Feyensv Feyensv commented Jul 1, 2022

  • Bump Sphinx version to 4.3.2 (default for Debian Jammy).
  • Remove the retrocompatibility for Sphinx < 3.5 (warnings are raised in
    4.5).
  • Fix translator issue crashing builds.

Task - 2898477

@robodoo
Copy link
Collaborator

robodoo commented Jul 1, 2022

@Feyensv Feyensv marked this pull request as ready for review July 11, 2022 13:58
@Feyensv Feyensv requested a review from AntoineVDV July 11, 2022 13:59
@C3POdoo C3POdoo requested a review from a team July 11, 2022 14:05
@Feyensv Feyensv force-pushed the 15.0-sphinx-bump-vfe branch from 55f00d4 to 3a17ea7 Compare July 12, 2022 09:49
@AntoineVDV AntoineVDV force-pushed the 15.0-sphinx-bump-vfe branch from 3a17ea7 to 3f5aec4 Compare July 12, 2022 14:15
@AntoineVDV AntoineVDV changed the title [FIX] core: allow building doc in python 3.10 [IMP] conf.py, requirements.txt, *: allow building with Python 3.10 Jul 12, 2022
@AntoineVDV AntoineVDV force-pushed the 15.0-sphinx-bump-vfe branch from 3f5aec4 to e7b333d Compare July 12, 2022 14:39
@AntoineVDV
Copy link
Collaborator

@fw-bot up to saas-15.4

@fw-bot
Copy link
Collaborator

fw-bot commented Jul 12, 2022

Forward-porting to 'saas-15.4'.

@AntoineVDV AntoineVDV force-pushed the 15.0-sphinx-bump-vfe branch from e7b333d to 3618e17 Compare July 14, 2022 11:55
Feyensv added 2 commits July 14, 2022 14:55
- Bump Sphinx version to 4.3.2 (default for Debian Jammy).
- Remove the retrocompatibility for Sphinx < 3.5 (warnings are raised in
  4.5).
- Fix translator issue crashing builds.
- Unpin jinja2 from the requirements as the base issue is solved in sphinx 4

Task - 2898477
The `file` option for the `csv-table` directive expects relative file
paths rather than absolute file paths since Sphinx 4. Since this was the
only usage of the `ODOO_ABSPATH` placeholder, it is removed with this
commit.

Task - 2898477
@Feyensv Feyensv force-pushed the 15.0-sphinx-bump-vfe branch from 3618e17 to 79cbc92 Compare July 14, 2022 12:55
@Feyensv
Copy link
Collaborator Author

Feyensv commented Jul 14, 2022

@robodoo r+ rebase-ff

@robodoo
Copy link
Collaborator

robodoo commented Jul 14, 2022

Merge method set to rebase and fast-forward

robodoo pushed a commit that referenced this pull request Jul 14, 2022
- Bump Sphinx version to 4.3.2 (default for Debian Jammy).
- Remove the retrocompatibility for Sphinx < 3.5 (warnings are raised in
  4.5).
- Fix translator issue crashing builds.
- Unpin jinja2 from the requirements as the base issue is solved in sphinx 4

Task - 2898477

Part-of: #2362
@robodoo robodoo closed this Jul 14, 2022
@robodoo robodoo closed this in 60adfac Jul 14, 2022
@robodoo robodoo temporarily deployed to merge July 14, 2022 16:42 Inactive
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.

4 participants