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

configure: update to latest config.guess and config.sub #10

Merged
merged 101 commits into from
Jul 23, 2024

Conversation

github-actions[bot]
Copy link

@github-actions github-actions bot commented Jul 3, 2024

This updates config.guess and config.sub to their latest versions.
If the two files are deleted in this PR, please check the logs of the workflow here:
Workflow run summary

Automated changes by create-pull-request GitHub action

a0x8o and others added 30 commits July 3, 2024 10:36
* CI: add nix package and dev environment test

* nix: exclude more files from package source

---------

Co-authored-by: Edouard Choinière <[email protected]>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…SGeo#3948)

* style: Fixes unnecessary-collection-call (C408) for remaining code

Only applies fixes to non-empty collections by  `ruff check --select "C408" --unsafe-fixes --output-format=concise --fix` in order to limit the review scope.

* style: Apply black formatting

Separated in another commit for easier review of only the original changes without formatting

* style: Fixes unnecessary-collection-call (C408) for ctypesgen

Only applies fixes to non-empty collections by  `ruff check --select "C408" --unsafe-fixes --output-format=concise --fix` in order to limit the review scope.

* style: Fixes unnecessary-collection-call (C408) for gui/wxpython/psmap/instructions

* style: Adjust comment in gui/wxpython/psmap/instructions

* style: Remove comment in gui/wxpython/psmap/instructions
Concerns Pylint rule "consider-using-from-import / R0402"

Using `ruff check --output-format=concise --select PLR0402 --fix`.
…3947)

* style: Fixes unnecessary-collection-call (C408) for testsuite

Only applies fixes to non-empty collections by  `ruff check --select "C408" --unsafe-fixes --output-format=concise --fix --config 'include = ["*/testsuite/**.py"]'` in order to limit the review scope.

* style: Apply black formatting

Separated in another commit for easier review of only the original changes without formatting
* style(gui/wxpython): Fixes literal-membership (PLR6201)

Concerns Pylint rule "use-set-for-membership / R6201"

Using `ruff check --output-format=concise --select PLR6201 --preview --unsafe-fixes --fix gui/`.

* style: Apply single element in set formatting
Concerns Pylint rule "use-set-for-membership / R6201"

Using `ruff check --output-format=concise --select PLR6201 --preview --unsafe-fixes --fix`, but reverting some changes where it wasn't clear if it was safe.
Using  `ruff check --select "PLR1722" --unsafe-fixes --output-format=concise --fix`.
When building the image, only PROJ 8.2.1 is available, and PDAL 2.7.2 checks for PROJ 9.0+. Reverting until the Dockerfile is modified to include PROJ 9.0+
* style(temporal): Sort and group imports

Uses a combination of `ruff check --output-format=concise --select I --fix 'temporal/' 'python/grass/temporal'`, `isort --profile=black temporal python/grass/temporal` and `black .`

* style(temporal): Sort and group imports for __init__.py
…01) (OSGeo#3943)

* style: Fix single-line-implicit-string-concatenation violations (ISC001)

Only applies safe fixes with `ruff check --select "ISC001" --fix` in order to limit the review scope.

* style: Apply black formatting

Separated in another commit for easier review of the changes of the first commit
Concerns Pylint rule "unnecessary-lambda / W0108"

Using `ruff check --output-format=concise --select PLW0108 --preview --fix --unsafe-fixes`.

Didn't apply to cases where it wasn't clear if it was safe
Use parson to add json output format support to the r.profile module.
* Rename l to line when using `f.readlines()`.

* Remove "l"s from parameters since they aren't used.

* Rename "l" to "lx" to assign a lexer object.

* Add "# noqa: E741" as renaming parameters won't be applicable in this case.

* Rename "l" to "ln" as line number.

* Rename "l" to "layer" when looping through the layer list.

* Remove E741 for files in the 'locale', `python`, and `scripts` directory.

* Rename "ln" to "line_number".
* Add JSON support to r.univar

Use parson to add json output format support to the r.univar module.

* Apply suggestions from code review

---------

Co-authored-by: Nicklas Larsson <[email protected]>
* Add JSON support to v.univar

Use parson to add json output format support to the v.univar module.

* update percentile format

* rename test file
…o#3979)

Reduce scope of minreg and maxreg variables in d.labels/main.c

Co-authored-by: Shubham Vasudeo Desai <[email protected]>
Reduce scope of 'bg_w' variable in d.legend.vect/draw.c

Co-authored-by: Shubham Vasudeo Desai <[email protected]>
…eo#3975)

Fix uninitialized variable 'encoding' in parser_interface.c

Co-authored-by: Shubham Vasudeo Desai <[email protected]>
* CI(ruff): Fix typo in ruff suggestions

* CI(ruff): Upload suggestions even if not all linting errors were fixed

* CI(ruff): Try suggesting more changes

* CI(ruff): Upload suggestions even if not all linting errors were fixed but do not run when cancelled
…grass.script as gs`) (OSGeo#3981)

* style: Unignore unconventional-import-alias (ICN001) ruff rule

* style: Use convention `import matplotlib as mpl`

* style: Use convention `import numpy as np`

* style: Use convention `import grass.script as gs`

* test: Remove unused grass.script in test file

* style: Add missing alias to `import grass.script as gs`

* style: Alias `import grass.script as gs` when already imported at file-level

* Remove duplicated grass.script lazy import in wxpython/gui_core/ghelp.py
a0x8o and others added 4 commits July 23, 2024 11:22
If any input cell from imagery group is NULL, the result of classification is also NULL. The implementation of original cat restoration (79f9500) did not check for this corner case and thus caused an out of bounds value access.
Replaces star import (from x import *) by regular imports (from x import y).

Enables the relevant Flake8 warnings.

Adds the warning to per-file ignores for init files where we expect and allow
using star imports (unfortunately some init files contain actual code suffers
from these warning being disabled, but that's a different issue).
properly terminate input WKT string

Co-authored-by: Marc Jansen <[email protected]>
@a0x8o a0x8o merged commit e4faab9 into main Jul 23, 2024
2 of 18 checks passed
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.

8 participants