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

Cap documenter version to "0" #37

Merged
merged 1 commit into from
Oct 31, 2023

Conversation

j-fu
Copy link
Contributor

@j-fu j-fu commented Oct 3, 2023

There seem to be breaking changes in Documenter 1.x regarding Documenter.Expanders and code block highlighting in docstrings.

This PR proposes to limit the Documenter version to <1.0 as a quick fix until there is time to figure out what has been changed.

Alternatively, we could suspend highlighting in docstrings.

There seem to be breaking changes in Documenter 1.x regarding Documenter.Expanders
and code block highlighting in docstrings.

This PR proposes to limit the Documenter version to <1.0 as a quick fix
until there is time to figure out what has been changed.

Alternatively, we could suspend highlighting in docstrings.
j-fu referenced this pull request Oct 3, 2023
* Fix print_script for 1.10

Appearantly, at the time when @generated is evaluated, methods from
other packages may not have been compiled yet, so
`hasmethod(show, Tuple{IO, MIME{Symbol("text/javascript")}, value})`
returns false, and the @generated function print_script throws an error.

As the old version worked in 1.9.3, this appears to be the result of
subtle changes in precompilation/initialization
behavior somewhere between 1.9.3 and 1.10.0-beta2.

* Add version discriminator for print_script(::IO, ::MIME,::Any)
Keep the old behavior (via @generated) for versions <=1.9.x.
@j-fu
Copy link
Contributor Author

j-fu commented Oct 10, 2023

Just tried to figure out if I could fix the highlighting instead for the new Documenter, but
the Documenter release notes state:

For upgrading: To keep using the Markdown backend, refer to the DocumenterMarkdown package. That package might not immediately support the latest Documenter version, however.

This is indeed still the case.

so I'd like to keep this PR going as it is together with a new release to be able to use #36 in Pluto notebooks.

@j-fu
Copy link
Contributor Author

j-fu commented Oct 16, 2023

Bump this to finally resolve #35

@fonsp fonsp merged commit 5005d05 into JuliaPluto:master Oct 31, 2023
@j-fu
Copy link
Contributor Author

j-fu commented Oct 31, 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