Skip to content

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
@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