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

Change consolidated fields code for docutils 0.22 #835

Open
tristanlatr opened this issue Nov 15, 2024 · 1 comment
Open

Change consolidated fields code for docutils 0.22 #835

tristanlatr opened this issue Nov 15, 2024 · 1 comment
Labels

Comments

@tristanlatr
Copy link
Contributor

From the release notes:

Document Tree / Docutils DTD
Allow multiple elements in a <definition_list_item> (third-party writers may need adaption).

Since the code that handles the consolidated fields is just a post processor for definition lists, we should make sur it does not fail when hitting several terms in the same item.

@tristanlatr
Copy link
Contributor Author

After a quick review of the code that processes definitions lists, I believe it could cause a undefined behavior when there are two terms and no classifier: more specifically I think the second term could be interpreted as being the type of the argument when should instead fail to parse the line (the type should only be specified with the classifier).

@tristanlatr tristanlatr changed the title Investigate whether we need to change some code for docutils 0.22 Change consolidated fields code for docutils 0.22 Nov 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant