-
-
Notifications
You must be signed in to change notification settings - Fork 18.1k
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
DOC: Fix doc warning "Code input with no code" #26788
Comments
I'm on it 👍 |
You need to substract 15, then the number matches the line numbers in the rst files. That's because of the header that gets injected (something I mentioned at that point as annoying for the header approach, but I don't really have a good alternative) |
I noticed a couple had In a tiny example, |
|
I think a custom directive may be able to handle the header throwing off of
line numbers, but that's quite a bit more complex.
…On Tue, Jun 11, 2019 at 11:07 AM Joris Van den Bossche < ***@***.***> wrote:
Also, the reported lines aren't probably correct. We should research why
this is failing, and fix the problem.
You need to substract 15, then the number matches the line numbers in the
rst files. That's because of the header that gets injected (something I
mentioned at that point as annoying for the header approach, but I don't
really have a good alternative)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#26788?email_source=notifications&email_token=AAKAOIQ4PPWQJMJLP3N2ENTPZ7EVHA5CNFSM4HW7XAV2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXNU4LA#issuecomment-500911660>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAKAOISVH4M7NSH62D7T6FLPZ7EVHANCNFSM4HW7XAVQ>
.
|
BTW, the issue for the problem with the IPython directive is here: ipython/ipython#11362 |
xref #24173
When building the documentation
./doc/make.py html
, among other warnings we find the next:All code blocks seem to have code, so it's not obvious why we get the issues. Also, the reported lines aren't probably correct. We should research why this is failing, and fix the problem.
The text was updated successfully, but these errors were encountered: