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

markdown version fix #459

Merged
merged 1 commit into from
Jul 22, 2022
Merged

Conversation

ipdemes
Copy link
Contributor

@ipdemes ipdemes commented Jul 19, 2022

I had to add upper version for markdown due to some breaking changes to it that are not yet supported in the latest release of the sphinx-markdown-tables.
Please see more details here: ryanfox/sphinx-markdown-tables#36

@ipdemes ipdemes force-pushed the markdown_version_fix branch from c527b79 to 8404e5c Compare July 19, 2022 17:24
@ipdemes ipdemes requested review from bryevdv and manopapad July 19, 2022 17:24
@bryevdv
Copy link
Contributor

bryevdv commented Jul 19, 2022

Since it looks like there might be a fix released soon, and since not many folks seem to build the docs locally, I think doing nothing is also an acceptable short term solution, until the problem fixes itself. :) Otherwise, LTGM but ideally there would be a corresponding change in legate.core to keep the env files in sync.

@ipdemes
Copy link
Contributor Author

ipdemes commented Jul 20, 2022

@bryevdv I think it would be safe to keep this restriction for some time. When new version of sphinx-markdown-tables is available, we will have to put a minimum version restriction on it.

@ipdemes ipdemes merged commit a360059 into nv-legate:branch-22.07 Jul 22, 2022
jjwilke pushed a commit to jjwilke/cunumeric that referenced this pull request Jul 29, 2022
jjwilke pushed a commit to jjwilke/cunumeric that referenced this pull request Jul 29, 2022
@ipdemes ipdemes deleted the markdown_version_fix branch August 2, 2022 17:33
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