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

common mechanism for parsing docutils afield_lists to dicts` #77

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

weefuzzy
Copy link
Member

As broached in #75 we may well nest the seealso blocks which would be enough to spur me into doing this common task using a uniform mechanism

@weefuzzy weefuzzy requested a review from jamesb93 February 10, 2022 13:55
@jamesb93
Copy link
Member

Do you want me to test if it breaks under some use?

@weefuzzy
Copy link
Member Author

There's a test in the PR, so I'm relatively happy that it works in the abstract. Whether we'll actually need it comes down to what collective decision we come to about how to express seealsos of various stripes in the docs

@weefuzzy weefuzzy changed the base branch from dev to main February 21, 2023 09:02
@jamesb93
Copy link
Member

Not sure how this slipped under the radar @weefuzzy. Do we want to do something with this?

@weefuzzy
Copy link
Member Author

Not sure how this slipped under the radar @weefuzzy. Do we want to do something with this?

Looks like we never revisited the original motivating question (doing stuff with seealso). Still, if we revisit that, this will probably be useful.

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

Successfully merging this pull request may close these issues.

2 participants