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

Adjust the RTD dependencies for old sphinx version #387

Closed
wants to merge 1 commit into from

Conversation

mbeacom
Copy link
Contributor

@mbeacom mbeacom commented Jan 10, 2023

Closed to re-create PR for RTD PR-specific builds.
Issue #, if available: -

Description of changes: The goal of this PR is to resolve the RTD build failures (similar to here: https://readthedocs.org/projects/aws-config-rdk/builds/19116212/) -- the encountered failure is described here: readthedocs/readthedocs.org#9037

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@mbeacom mbeacom self-assigned this Jan 10, 2023
@@ -1,5 +1,3 @@
# -*- coding: utf-8 -*-
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This encoding is deprecated and superfluous in newer python minors 3.7+

@@ -23,7 +21,7 @@
# -- Project information -----------------------------------------------------

project = "RDK"
copyright = "2017-2022 Amazon.com, Inc. or its affiliates. All Rights Reserved"
copyright = "2017-2023 Amazon.com, Inc. or its affiliates. All Rights Reserved"
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Realistically, the copyright could simply remain the initial creation date, but since a range was used... let's update it.

@@ -3,4 +3,4 @@ sphinx-argparse==0.2.5
sphinx-rtd-theme==0.4.3
sphinxcontrib-websupport==1.1.0
PyYAML==5.4.1

jinja2<3.1.0
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should resolve the build failures due to jinja2.

@mbeacom mbeacom closed this Jan 10, 2023
@mbeacom mbeacom removed their assignment Jan 10, 2023
@mbeacom mbeacom deleted the fix-rtd branch January 10, 2023 14:38
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.

1 participant