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

fix warning for unexpected param #609

Conversation

Artur-Galstyan
Copy link
Contributor

Removes the "Unexpected argument" warning.

@patrick-kidger
Copy link
Owner

What's the warning you're getting?

@Artur-Galstyan
Copy link
Contributor Author

This one:
image

(I'm using PyCharm)

@patrick-kidger
Copy link
Owner

It looks like this is a PyCharm bug: https://youtrack.jetbrains.com/issue/PY-54560
(Perhaps consider switching to an alternative like VSCode / Vim / Helix / etc.)

On balance I'm inclined not to include this here -- adding __init__ methods to all classes which currently use the default __init__ is a lot of extra noise.

@Artur-Galstyan
Copy link
Contributor Author

I see, indeed. Alright, we can close this PR then.

(That's too bad, I specifically chose PyCharm because it has the best Vim implementation, but its autocompletion and intellisense aren't as good as hoped. Back to good ol' Neovim it is then)

@Artur-Galstyan Artur-Galstyan deleted the fix-nn-lambda-warning branch December 4, 2023 17:56
@patrick-kidger
Copy link
Owner

Haha, sorry!
FWIW allow me to recommend Helix, which is vim-like whilst being "batteries included": LSP, tree-sitter, file explorer etc. I'm a big fan!

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

Successfully merging this pull request may close these issues.

2 participants