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

set IO context for doctest output relative to sandbox module #848

Merged
merged 1 commit into from
Oct 2, 2018

Conversation

JeffBezanson
Copy link
Contributor

If we make a change like JuliaLang/julia#29466, this will be needed. Should be safe now as well.

@mortenpi mortenpi added this to the 0.20.0 milestone Oct 2, 2018
@mortenpi mortenpi merged commit 98d5c59 into master Oct 2, 2018
@mortenpi mortenpi deleted the jb/ioctx branch October 2, 2018 20:31
mortenpi pushed a commit that referenced this pull request Oct 2, 2018
@mortenpi
Copy link
Member

mortenpi commented Oct 2, 2018

LGTM, thanks!

I would suggest bumping Documenter in JuliaLang/julia#29470 and merging that to get this onto master. @fredrikekre?

But I will also backport this to 0.19.

@mortenpi mortenpi modified the milestones: 0.20.0, 0.19.7 Oct 2, 2018
@mortenpi mortenpi mentioned this pull request Oct 3, 2018
mortenpi pushed a commit that referenced this pull request Oct 3, 2018
Unlike the original patch, this creates a local unwrapiocontext function
that is necessary for this to work on 0.6, as Base.unwrapcontext was
introduce in 0.7.

(cherry picked from commit 98d5c59)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants