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

PR: Add additional cases for code cells and block comments #10

Merged
merged 5 commits into from
Nov 23, 2020

Conversation

andfoy
Copy link
Member

@andfoy andfoy commented Nov 12, 2020

Fixes #8
Fixes #9

@andfoy andfoy added this to the v0.1.2 milestone Nov 12, 2020
@andfoy andfoy self-assigned this Nov 12, 2020
@andfoy
Copy link
Member Author

andfoy commented Nov 12, 2020

@jnsebgosselin @eafpres @ccordoba12, could you please check if this PR address the missing code cells/block comments syntax?

@jnsebgosselin
Copy link
Member

So based on what was said in PR spyder-ide/spyder#13885, if I want to test that, I need to clone the repo locally, fetch the branch and install the repo with pip install -e . ?

Is there any risk that I'm going to break my conda installation by doing that? If I recall, Carlos always said that mixing pip and conda is something that is very bad to do.

@andfoy
Copy link
Member Author

andfoy commented Nov 12, 2020

@jnsebgosselin, since this package does not depend on Spyder, it shouldn't break anything, so you can install it with pip without any problems, just make sure that is installed with pip install -e . --no-deps

@jnsebgosselin
Copy link
Member

@jnsebgosselin, since this package does not depend on Spyder, it shouldn't break anything, so you can install it with pip without any problems, just make sure that is installed with pip install -e . --no-deps

Ok excellent. The --no-deps part is good. I'm going to try it out later today.

@eafpres
Copy link

eafpres commented Nov 12, 2020

@jnsebgosselin @eafpres @ccordoba12, could you please check if this PR address the missing code cells/block comments syntax?

Honestly I'm not sure I'm competent to test this without breaking something locally, so please do not wait on me.

@andfoy
Copy link
Member Author

andfoy commented Nov 12, 2020

@eafpres Don't worry, just a quick question should # ### (note the space before the 3 #'s) be a block comment as well?

@eafpres
Copy link

eafpres commented Nov 13, 2020

question should # ### (note the space before the 3 #'s) be a block comment as well?

I can understand that with the # %% case but this makes less sense to me. However, I’d rather defer to the dev team on that.

Copy link
Member

@ccordoba12 ccordoba12 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks pretty good, thanks @andfoy!

pyls_spyder/plugin.py Outdated Show resolved Hide resolved
pyls_spyder/plugin.py Outdated Show resolved Hide resolved
pyls_spyder/plugin.py Show resolved Hide resolved
pyls_spyder/utils.py Show resolved Hide resolved
@andfoy
Copy link
Member Author

andfoy commented Nov 23, 2020

@ccordoba12, all the review comments should be fixed now

Copy link
Member

@ccordoba12 ccordoba12 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Last comment for you @andfoy, then this should be ready.

pyls_spyder/tests/test_plugin.py Outdated Show resolved Hide resolved
Co-authored-by: Carlos Cordoba <ccordoba12@gmail.com>
Copy link
Member

@ccordoba12 ccordoba12 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @andfoy!

@MCilento93
Copy link

@ccordoba12, all the review comments should be fixed now

Hello
Which release has the issue of review comments fixed?

@andfoy
Copy link
Member Author

andfoy commented Nov 25, 2020

Hi @MCilento93, this fix is available in pyls-spyder v0.2.1, released yesterday

@MCilento93
Copy link

Mmm I have spyder 4.2.0 o 4.1.5

Can I install or do I need to wait for next spyder release?

Sincerely

@andfoy
Copy link
Member Author

andfoy commented Nov 25, 2020

You can install it by using conda or pip:

# Anaconda (Recommended)
conda install pyls-spyder=0.2.1 -c spyder-ide

# Pip
pip install -U pyls-spyder

@MCilento93
Copy link

You can install it by using conda or pip:

# Anaconda (Recommended)
conda install pyls-spyder=0.2.1 -c spyder-ide

# Pip
pip install -U pyls-spyder

installed, but unfortunately I do not see the comments row (like ### , !!! and so forth)

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