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

wxGUI: Fixed bare exceptions in mapdisp/ #4979

Closed
wants to merge 5 commits into from

Conversation

arohanajit
Copy link
Contributor

Added ValueError for string/split operations and IndexError for indexing

Arohan Ajit and others added 2 commits January 23, 2025 11:34
@echoix echoix changed the title wxGUI: FIxed bare exceptions in mapdisp/ wxGUI: Fixed bare exceptions in mapdisp/ Jan 23, 2025
@github-actions github-actions bot added GUI wxGUI related Python Related code is in Python labels Jan 23, 2025
Copy link
Contributor

@petrasovaa petrasovaa left a comment

Choose a reason for hiding this comment

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

There are unrelated changes in the PR.

.flake8 Outdated Show resolved Hide resolved
Copy link
Contributor

Choose a reason for hiding this comment

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

Unrelated changes

from ctypes import CFUNCTYPE, byref, c_double, c_int, pointer
from grass.lib.ctypes_preamble import UNCHECKED, String

import wx
from core.debug import Debug
from core.gcmd import DecodeString
from core.settings import UserSettings

try:
Copy link
Contributor

Choose a reason for hiding this comment

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

[black] reported by reviewdog 🐶

Suggested change
try:
try:

@arohanajit arohanajit closed this Jan 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
GUI wxGUI related Python Related code is in Python
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants