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.

Sign up for GitHub

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

Jump to bottom

How is sphinxcontrib-httpdomain used in the project? #1533

Closed
befeleme opened this issue Oct 17, 2023 · 1 comment
Closed

How is sphinxcontrib-httpdomain used in the project? #1533

befeleme opened this issue Oct 17, 2023 · 1 comment
Labels
Bug A bug Needed: replication Bug replication is required

Comments

@befeleme
Copy link
Contributor

Problem

In setup.cfg there is a dev extra defined with sphinxcontrib-httpdomain as one of the dependencies.

sphinx_rtd_theme/setup.cfg

Line 57 in 8ce23ce

sphinxcontrib-httpdomain

I can't find it being used anywhere in the project. Is it needed?
@befeleme befeleme added Bug A bug Needed: replication Bug replication is required labels Oct 17, 2023
@humitos
Copy link
Member

humitos commented Aug 12, 2024

This dependency was removed.

@humitos humitos closed this as completed Aug 12, 2024
humitos added a commit that referenced this issue Aug 12, 2024
@humitos
Remove unused dependency
d1d1d97 
Closes #1533
humitos added a commit that referenced this issue Aug 20, 2024
@humitos
Prepare for 3.0 release (#1576)
6b587d1 
* Tests: `sphinxdev` tox environment installs Sphinx from `master`

It seems we had a bug in the name and it wasn't installing Sphinx from `master`.
I expect this test to fail now because we pin `Sphinx<8`, but that's fine for now.

* Prepare for 3.0 release

- Drop Python <3.8
- Drop docutils <=0.18
- Add support for docutils 0.21
- Add support for Python 3.12

This follows the plan we wrote in
https://sphinx-rtd-theme.readthedocs.io/en/stable/development.html#roadmap-release-3-0-0

The main idea here is to move forward with newer versions and drop support for
old stuffs so we reduce the maintanence burden.

* Update CircleCI matrix

* More fixes to CircleCI

* Run test over Sphinx 8.0

* Update contributing guide

* Remove deprecated `readthedocs-sphinx-ext`

* Remove old Sphinx versions

* Uninstall our extension

* Show a warning if `extra_css_files` is in the `html_context`

See #450

* Update development page

* Add deprecate warning for `analytics_` theme options

* Update docs

* Remove unused dependency

Closes #1533

* Remove environment variables from our docs

This was already added in a more generic way to the theme itself.

* Require Sphinx >=6.0

* Deprecation warning about `get_html_theme_path`

See #1479 (comment)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug A bug Needed: replication Bug replication is required
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@humitos @befeleme