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

checks: Pylint 3 config #143

Closed
wants to merge 28 commits into from
Closed

checks: Pylint 3 config #143

wants to merge 28 commits into from

Conversation

echoix
Copy link
Owner

@echoix echoix commented Jun 29, 2024

No description provided.

@echoix echoix force-pushed the pylint-3-config branch 2 times, most recently from 42ea94a to e1b862f Compare June 30, 2024 12:49
@echoix echoix changed the title Pylint 3 config checks: Pylint 3 config Jun 30, 2024
Repository owner deleted a comment from github-actions bot Jun 30, 2024
Repository owner deleted a comment from github-actions bot Jun 30, 2024
Repository owner deleted a comment from github-actions bot Jun 30, 2024
@echoix echoix force-pushed the pylint-3-config branch 4 times, most recently from d0bc70a to f6f0e65 Compare July 1, 2024 00:47
@echoix echoix force-pushed the pylint-3-config branch 2 times, most recently from 8da0f2e to 45d8bb0 Compare July 1, 2024 02:50
DEBUG: VSCode settings clean pure json syntax errors
…tput variable

Pylint 3.2.6 error: R0204: Redefinition of output type from bytes to grass.script.utils.KeyValue (redefined-variable-type)

Adds typing to the function analyzed, and also to the return type of the gs.parse_key_val() used.
DEBUG: Reset Pylint config to defaults
python: Add some typing to gui/wxpython/core/gcmd.py CommandThread

LiteralString is only available since Python 3.11, so use str instead
The runtime type is main_window.frame.GMFrame, for both mapdisp.frame.MapPanel and nviz.mapwindow.GLWindow
The runtime type can also be lmgr.frame.GMFrame, for both mapdisp.frame.MapPanel and nviz.mapwindow.GLWindow when using single window mode
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant