Skip to content

Change recommonmark for myst#1596

Closed
scarlehoff wants to merge 1 commit into
masterfrom
change_recommonmark_to_myst
Closed

Change recommonmark for myst#1596
scarlehoff wants to merge 1 commit into
masterfrom
change_recommonmark_to_myst

Conversation

@scarlehoff
Copy link
Copy Markdown
Member

While it might not be the problem of #1537, this is something that we should've done some time ago given #1220 (comment)

I think that the documentation produced by this branch is equal to the documentation in master (actually, a bit better because I've fixed a few broken links I've found) but I've only checked by eye.

@Zaharid
Copy link
Copy Markdown
Contributor

Zaharid commented Sep 8, 2022

Since we are touching the md files anyway, how bad would it be to convert them to rst as per the original comment? Supporting the two things was quickly disliked by everyone, hence the note here

https://docs.nnpdf.science/contributing/sphinx-documentation.html?highlight=rst

@scarlehoff
Copy link
Copy Markdown
Member Author

scarlehoff commented Sep 8, 2022

how bad would it be to convert them to rst as per the original comment?

If recommonmark is the cause of #1537, very bad (because it would take time)

(if not, there's no hurry and it would be nice indeed)

@scarlehoff
Copy link
Copy Markdown
Member Author

Closing in favour of #1597

@scarlehoff scarlehoff closed this Sep 14, 2022
@scarlehoff scarlehoff deleted the change_recommonmark_to_myst branch September 14, 2022 13:51
@Zaharid Zaharid mentioned this pull request Feb 16, 2023
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