forked from OSGeo/grass
-
Notifications
You must be signed in to change notification settings - Fork 0
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
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* 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
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]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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