Skip to content

Conversation

henryiii
Copy link
Collaborator

Closes #559.

I have a couple of followup ideas, but this should at least get the docs building again.

@henryiii
Copy link
Collaborator Author

Not seeing a RtD PR build.

@dhermes
Copy link
Collaborator

dhermes commented Jan 11, 2022

Not seeing a RtD PR build.

Ditto; I unfortunately don't have admin on this repo so can't check if the webhook is installed

Copy link
Collaborator

@dhermes dhermes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM; let's just merge and see if it resolves

@henryiii
Copy link
Collaborator Author

Should be quite safe, it just might not fix the problem. :)

@henryiii
Copy link
Collaborator Author

henryiii commented Jan 11, 2022

(Even if the web hook was enabled, sometimes if it was old it might need to be refreshed - I feel like I've had to fight with it before)

If this works, most issues should be caught by the GHA docs build (except this one). You normally just lose the ability to preview docs.

@dhermes dhermes merged commit 04ad575 into wntrblm:main Jan 11, 2022
@dhermes
Copy link
Collaborator

dhermes commented Jan 11, 2022

@dhermes dhermes mentioned this pull request Jan 11, 2022
@henryiii
Copy link
Collaborator Author

Ahh, I almost replaced recommonmark with myst-parser (it's deprecated), but then I left it for followup. Looks like that's what triggering it. Instead of looking up the version of recommonmark needed, let's just go ahead and upgrade to myst.

@henryiii henryiii deleted the patch-1 branch January 11, 2022 22:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

readthedocs broken
2 participants