[GRASS-git] [OSGeo/grass] 9e1e85: style: Fix if-else-block-instead-of-if-exp (SIM108...

[Edouard Choinière]

  Branch: refs/heads/main
  Home:   https://github.com/OSGeo/grass
  Commit: 9e1e85d6079f36962f19b32793b31f43c23b5a4f
      https://github.com/OSGeo/grass/commit/9e1e85d6079f36962f19b32793b31f43c23b5a4f
  Author: Edouard Choinière <27212526+echoix at users.noreply.github.com>
  Date:   2024-10-25 (Fri, 25 Oct 2024)

  Changed paths:
    M gui/wxpython/animation/dialogs.py
    M gui/wxpython/animation/frame.py
    M gui/wxpython/animation/temporal_manager.py
    M gui/wxpython/core/gcmd.py
    M gui/wxpython/core/gconsole.py
    M gui/wxpython/core/menutree.py
    M gui/wxpython/core/render.py
    M gui/wxpython/core/settings.py
    M gui/wxpython/core/treemodel.py
    M gui/wxpython/core/utils.py
    M gui/wxpython/core/workspace.py
    M gui/wxpython/datacatalog/tree.py
    M gui/wxpython/dbmgr/base.py
    M gui/wxpython/dbmgr/dialogs.py
    M gui/wxpython/dbmgr/sqlbuilder.py
    M gui/wxpython/gcp/manager.py
    M gui/wxpython/gmodeler/dialogs.py
    M gui/wxpython/gmodeler/model.py
    M gui/wxpython/gmodeler/panels.py
    M gui/wxpython/gui_core/dialogs.py
    M gui/wxpython/gui_core/forms.py
    M gui/wxpython/gui_core/gselect.py
    M gui/wxpython/gui_core/mapdisp.py
    M gui/wxpython/gui_core/menu.py
    M gui/wxpython/gui_core/preferences.py
    M gui/wxpython/gui_core/prompt.py
    M gui/wxpython/gui_core/treeview.py
    M gui/wxpython/iclass/dialogs.py
    M gui/wxpython/iclass/digit.py
    M gui/wxpython/image2target/ii2t_gis_set.py
    M gui/wxpython/image2target/ii2t_manager.py
    M gui/wxpython/iscatt/controllers.py
    M gui/wxpython/iscatt/core_c.py
    M gui/wxpython/iscatt/frame.py
    M gui/wxpython/iscatt/iscatt_core.py
    M gui/wxpython/lmgr/frame.py
    M gui/wxpython/lmgr/layertree.py
    M gui/wxpython/lmgr/menudata.py
    M gui/wxpython/location_wizard/wizard.py
    M gui/wxpython/main_window/frame.py
    M gui/wxpython/mapdisp/frame.py
    M gui/wxpython/mapdisp/toolbars.py
    M gui/wxpython/mapwin/buffered.py
    M gui/wxpython/mapwin/decorations.py
    M gui/wxpython/modules/colorrules.py
    M gui/wxpython/modules/extensions.py
    M gui/wxpython/modules/import_export.py
    M gui/wxpython/modules/mcalc_builder.py
    M gui/wxpython/nviz/mapwindow.py
    M gui/wxpython/nviz/tools.py
    M gui/wxpython/nviz/workspace.py
    M gui/wxpython/nviz/wxnviz.py
    M gui/wxpython/photo2image/ip2i_manager.py
    M gui/wxpython/psmap/dialogs.py
    M gui/wxpython/psmap/frame.py
    M gui/wxpython/psmap/utils.py
    M gui/wxpython/rdigit/controller.py
    M gui/wxpython/timeline/frame.py
    M gui/wxpython/tools/update_menudata.py
    M gui/wxpython/tplot/frame.py
    M gui/wxpython/vdigit/dialogs.py
    M gui/wxpython/vdigit/mapwindow.py
    M gui/wxpython/vdigit/preferences.py
    M gui/wxpython/vdigit/toolbars.py
    M gui/wxpython/vdigit/wxdigit.py
    M gui/wxpython/vdigit/wxdisplay.py
    M gui/wxpython/vnet/dialogs.py
    M gui/wxpython/vnet/vnet_data.py
    M gui/wxpython/vnet/vnet_utils.py
    M gui/wxpython/web_services/widgets.py
    M gui/wxpython/wxgui.py
    M gui/wxpython/wxplot/scatter.py
    M pyproject.toml
    M python/grass/script/core.py
    M python/grass/temporal/abstract_dataset.py

  Log Message:
  -----------
  style: Fix if-else-block-instead-of-if-exp (SIM108) (part 3) (#4570)

* style: Fix if-else-block-instead-of-if-exp (SIM108) in scripts/

Ruff rule: https://docs.astral.sh/ruff/rules/if-else-block-instead-of-if-exp

* style: Fix if-else-block-instead-of-if-exp (SIM108) in python/

Ruff rule: https://docs.astral.sh/ruff/rules/if-else-block-instead-of-if-exp

* checks: Rename inner variable shadowing type to _type in python/grass/temporal/gui_support.py

* style: Manual fixes for if-else-block-instead-of-if-exp (SIM108) in python/

Ruff rule: https://docs.astral.sh/ruff/rules/if-else-block-instead-of-if-exp

* style: Fix if-else-block-instead-of-if-exp (SIM108) in gui/

Ruff rule: https://docs.astral.sh/ruff/rules/if-else-block-instead-of-if-exp

* checks: Rename inner variable shadowing list to _list in gui/wxpython/core/render.py

* style: Manual fixes for if-else-block-instead-of-if-exp (SIM108) in gui/

Ruff rule: https://docs.astral.sh/ruff/rules/if-else-block-instead-of-if-exp

* python: Add type annotations for grass.script.core.parser()

* python: Add type annotations for is_time_absolute() and is_time_relative() in grass.temporal.AbstractDataset

* style: Enable Checking for SIM108

* Update pyproject.toml to remove fixed issues

* Update base.py

Co-authored-by: Anna Petrasova <kratochanna at gmail.com>

---------

Co-authored-by: Anna Petrasova <kratochanna at gmail.com>



To unsubscribe from these emails, change your notification settings at https://github.com/OSGeo/grass/settings/notifications