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

Restore support for list-style source_suffix with third-party parsers #12584

Merged
merged 2 commits into from
Jul 15, 2024

Conversation

AA-Turner
Copy link
Member

Fixes #12582

I have also added information/warning messages when source_suffix is mutated within Sphinx.

A

@AA-Turner
Copy link
Member Author

@jfbu @dbitouze does this fix things for you?

A

@jfbu
Copy link
Contributor

jfbu commented Jul 15, 2024

@jfbu @dbitouze does this fix things for you?

A

Yes it does! Thanks!

@jfbu
Copy link
Contributor

jfbu commented Jul 15, 2024

I have also added information/warning messages when source_suffix is mutated within Sphinx.

Seen:

Converting `source_suffix = ['.rst', '.md']` to `source_suffix = {'.rst': 'restructuredtext', '.md': 'restructuredtext'}`.

@AA-Turner AA-Turner merged commit 55c8953 into sphinx-doc:master Jul 15, 2024
21 checks passed
@AA-Turner AA-Turner deleted the fix-source-suffix branch July 15, 2024 11:45
@AA-Turner AA-Turner added this to the 7.4.x milestone Jul 17, 2024
Copy link
Contributor

@jfbu jfbu left a comment

Choose a reason for hiding this comment

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

Sorry I missed the missing space (spotted from having a look on transifex at chains to translate).

logger.warning(__("The config value `source_suffix' expects "
"a string, list of strings, or dictionary. "
"But `%r' is given." % source_suffix))
msg = __("The config value `source_suffix' expects a dictionary,"
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
msg = __("The config value `source_suffix' expects a dictionary,"
msg = __("The config value `source_suffix' expects a dictionary, "

@jfbu
Copy link
Contributor

jfbu commented Jul 25, 2024

ping @AA-Turner for missing space in message string...

@AA-Turner
Copy link
Member Author

Thanks! See 0cb3c07

A

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Sphinx 7.4.0 breaks parsing of MyST-Parser (and probably others) configuration in conf.py
2 participants