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

prepare for macrocall having a source location argument #480

Merged
merged 1 commit into from
May 15, 2017

Conversation

vtjnash
Copy link
Contributor

@vtjnash vtjnash commented May 10, 2017

JuliaLang/julia#21746

(will need JuliaLang/Compat.jl#355 to be merged and tagged first, then restart CI here)

@mortenpi
Copy link
Member

Since I'm not really familiar with the docsystem code -- this is an enhancement that will be available with 0.7 / master, right?

@vtjnash vtjnash force-pushed the jn/macro-sourcelocation branch from 234528d to b0f0b22 Compare May 15, 2017 15:06
@vtjnash
Copy link
Contributor Author

vtjnash commented May 15, 2017

Yes, this is for a PR against Julia master

@mortenpi
Copy link
Member

But 0.7 builds won't fail without this, right? I am wondering whether I should worry about backporting.

@mortenpi mortenpi added this to the 0.11 milestone May 15, 2017
@mortenpi mortenpi merged commit 27e8059 into JuliaDocs:master May 15, 2017
@vtjnash
Copy link
Contributor Author

vtjnash commented May 15, 2017

They'll start to fail once JuliaLang/julia#21746 is merged, which is why this needed to be done first

@vtjnash vtjnash deleted the jn/macro-sourcelocation branch May 15, 2017 22:53
@mortenpi
Copy link
Member

Ah, I'll tag a patch release for Documenter asap then.

@vtjnash
Copy link
Contributor Author

vtjnash commented May 15, 2017

Thanks :)

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