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

Work around #1698 #1699

Merged
merged 4 commits into from
Sep 22, 2021
Merged

Work around #1698 #1699

merged 4 commits into from
Sep 22, 2021

Conversation

ericphanson
Copy link
Contributor

@ericphanson ericphanson commented Sep 15, 2021

I think this would revert to the old behavior if we can't find the line info. Not sure why that is though-- seems like we'd need to know that to know how to repro the error for a test.

fixes #1698

src/DocTests.jl Outdated Show resolved Hide resolved
ericphanson and others added 3 commits September 16, 2021 01:02
Co-authored-by: Fredrik Ekre <ekrefredrik@gmail.com>
@fredrikekre fredrikekre merged commit a7cb9a6 into JuliaDocs:master Sep 22, 2021
@ericphanson ericphanson deleted the patch-1 branch September 22, 2021 09:43
@timholy
Copy link
Contributor

timholy commented Sep 30, 2021

Belated thanks to @ericphanson and @fredrikekre!

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.

MethodError: no method matching getindex(::Nothing, ::Int64)
4 participants