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

confoverrides are not respected for theme configuration #10120

Closed
chrisjsewell opened this issue Jan 19, 2022 · 5 comments
Closed

confoverrides are not respected for theme configuration #10120

chrisjsewell opened this issue Jan 19, 2022 · 5 comments

Comments

@chrisjsewell
Copy link
Member

chrisjsewell commented Jan 19, 2022

Describe the bug

Basically, there needs to be a slight amendment to #10079

cc @pradyunsg

How to Reproduce

I come across this when running tests with sphinx fixtures

pytest_plugins = 'sphinx.testing.fixtures'

@pytest.mark.sphinx('html', confoverrides={'pygments_dark_style': 'other'})
def test_html4_output(app, status, warning):
   # but pygments_dark_style wil not be set
    app.build()

Expected behavior

No response

Your project

furo

Screenshots

No response

OS

any

Python version

any

Sphinx version

4.3.2

Sphinx extensions

No response

Extra tools

No response

Additional context

No response

@pradyunsg
Copy link
Contributor

You need Sphinx 4.4.0. Can you reproduce with that?

@chrisjsewell
Copy link
Member Author

chrisjsewell commented Jan 19, 2022

You need Sphinx 4.4.0. Can you reproduce with that?

Yep: it picks up config in the conf.py (what the PR fixed) but not confoverrides (what the PR did not fix)

@AA-Turner AA-Turner added this to the some future version milestone Sep 29, 2022
@benjaoming
Copy link
Contributor

benjaoming commented Jan 24, 2023

I'm pretty sure that I have the same issue for a config value added to an extension. The conf value is added with app.add_config_value but defining it in confoverrides does not have an effect.

(using Sphinx 6.x)

@AA-Turner
Copy link
Member

See #10079.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 17, 2023
@AA-Turner AA-Turner reopened this Jan 4, 2024
@sphinx-doc sphinx-doc unlocked this conversation Jan 4, 2024
@AA-Turner
Copy link
Member

AA-Turner commented Jan 4, 2024

Overrides for theme configuration options (via --define, -D, etc) should now work properly following #11855.

A

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 4, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

4 participants