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

All ".. runblock:: pycon" blocks start with new line #8

Closed
lgarcin opened this issue Sep 10, 2017 · 9 comments
Closed

All ".. runblock:: pycon" blocks start with new line #8

lgarcin opened this issue Sep 10, 2017 · 9 comments
Labels

Comments

@lgarcin
Copy link

lgarcin commented Sep 10, 2017

First of all, thank for your work.
I noticed a bug in the output of runblock directives ; I think the title of the issue is self-explanatory.

For example

.. runblock:: pycon

    >>> a = 0
    >>> a += 1
    >>> a

produces


>>> a = 0
>>> a += 1
1

I'd like to get rid of the leading newline.

@WhyNotHugo
Copy link
Owner

Good catch, I'll try and look into this in the next days.

@WhyNotHugo WhyNotHugo added the bug label Sep 10, 2017
@endrebak
Copy link
Contributor

+1 on this.

endrebak added a commit to endrebak/sphinx-autorun-ebs that referenced this issue May 16, 2018
@endrebak
Copy link
Contributor

I uploaded a new version to PyPI called sphinx_autorun_ebs which I can personalize to my needs, but I do not mind making PRs here.

WhyNotHugo pushed a commit to endrebak/sphinx-autorun-ebs that referenced this issue May 16, 2018
WhyNotHugo pushed a commit that referenced this issue May 16, 2018
@torfsen
Copy link

torfsen commented Mar 2, 2019

I just ran into this in the current version (1.1.0). Could you prepare a new release which contains the bugfix?

@vaneseltine
Copy link

Yes, this has been fixed for a while -- it'd be great to have it released to PyPI.

@WhyNotHugo
Copy link
Owner

I'll try to get that pushed through during the day today :)

@WhyNotHugo
Copy link
Owner

Aw, dammit. The push fails. Looks like CI still also runs Python 3.3, gonna have to update that.

@WhyNotHugo
Copy link
Owner

Published for real this time!

@vaneseltine
Copy link

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants