-
Notifications
You must be signed in to change notification settings - Fork 17
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
Ambiguous readme/documentation #29
Comments
Thanks for the feedback! I'd be happy to accept a PR improving the docs
…On Thu, Apr 7, 2022, 18:57 omhland ***@***.***> wrote:
Hi!
I think the readme could do well with some extra information.
E.g. it is not clear what is meant by this let
g:cellmode_tmux_panenumber=\'0\'. Does this specify the pane number
running vim or python? I assumed it specified the pane containing vim
because consoles are generally below or to the right of the editors.
Second, I didn't get it working until using TMPDIR=$HOME/tmp vim which
was described in an issue. Maybe put this into the readme?
Note: I can submit 2 or 3 suggested modifications to the readme, if that
would make it easier for you.
Anyway, amazing plugin. Thx.
—
Reply to this email directly, view it on GitHub
<#29>, or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAD3V2Q4D2VZER6A6DBR33DVD4HYTANCNFSM5S2AFWMQ>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
Ok, have a look at my proposed changes |
Did you open a PR ? I can't seem to find it
…On Fri, Apr 8, 2022 at 1:24 PM Ole Haaland ***@***.***> wrote:
Ok, have a look at my proposed changes
—
Reply to this email directly, view it on GitHub
<#29 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAD3V2U6K66HYDC3HUEZNC3VEAJPPANCNFSM5S2AFWMQ>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hi!
I think the readme could do well with some extra information.
E.g. it is not clear what is meant by this
let g:cellmode_tmux_panenumber=\'0\'
. Does this specify the pane number running vim or python? I assumed it specified the pane containing vim because consoles are generally below or to the right of the editors.Second, I didn't get it working until using
TMPDIR=$HOME/tmp vim
which was described in an issue. Maybe put this into the readme?Note: I can submit 2 or 3 suggested modifications to the readme, if that would make it easier for you.
Anyway, amazing plugin. Thx.
The text was updated successfully, but these errors were encountered: