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

VS Code + pylint integration with p5 #66

Closed
zaha opened this issue Aug 9, 2018 · 4 comments · Fixed by #228
Closed

VS Code + pylint integration with p5 #66

zaha opened this issue Aug 9, 2018 · 4 comments · Fixed by #228

Comments

@zaha
Copy link

zaha commented Aug 9, 2018

Hey,

I've installed the lib on my system and copied the demo file. It runs well, but in VS Code with pylint it's giving me quite a lot of errors and warnings like:

  • E0102:function already defined line 1 (setup function)
  • E0102:function already defined line 1 (draw function)
  • E0602:Undefined variable 'mouse_is_pressed'
  • W0614:Unused import setup from wildcard import
  • and lots more

How can I set this up to play well with VS Code and pylint?
Sorry if this may sound dumb, I'm pretty new to python :)

@zaha zaha changed the title How to setup VS Code with pylint? How to setup VS Code and pylint to play well with this lib? Aug 9, 2018
@abhikpal
Copy link
Member

abhikpal commented Aug 9, 2018

I personally don't really know how pylint works nor do I regularly use VS Code. I don't really know how to make p5 pylint-compatible. And I don't know when I'll get the time to look into this.

It would be great if someone's more familiar with pylint takes this up : ) I've tagged this as "help-wanted"

@abhikpal abhikpal changed the title How to setup VS Code and pylint to play well with this lib? VS Code + pylint integration with p5 Aug 9, 2018
@zaha
Copy link
Author

zaha commented Aug 10, 2018

Great, thanks for the quick response. Hopefully somebody knows :)

btw, great library. I completely agree with your idea that Python is better suited for this than Java. Really hope the missing functionality is added soon as well! 👍

@jeremydouglass
Copy link
Contributor

Pylint is configured locally with a .pylintrc file in the root directory of the project being lint-ed.

I encountered a related problem using pylint with Processing.py .pyde projects. Here is a gist with the .pylintrc file that I added to projects directories to make them meaningfully lintable. It is largely taken from the Processing API. The config for p5py would be different -- but, I imagine, highly related.

https://gist.github.com/jeremydouglass/bc2193518a29cbf280d0cfc7415a9c82

@MrGallo
Copy link

MrGallo commented Oct 25, 2018

Very sorry, I referenced the wrong issue. I meant to reference #69.

ziyaointl pushed a commit that referenced this issue Aug 15, 2020
Added vscode and pylinter integration.

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

Successfully merging a pull request may close this issue.

4 participants