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.

Sign up for GitHub

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

Jump to bottom

style(scripts): Sort and group imports #3963

Closed
wants to merge 1 commit into from
Closed

style(scripts): Sort and group imports #3963

wants to merge 1 commit into from

Conversation

echoix
Copy link
Member

@echoix echoix commented Jul 1, 2024

Uses a combination of ruff check --output-format=concise --select I --fix scripts, isort --profile=black scripts and black .

Continues on the same PRs as #3961, #3959, and #3962
@echoix
style(scripts): Sort and group imports
af421af 
Uses a combination of `ruff check --output-format=concise --select I --fix scripts`, `isort --profile=black scripts` and `black .`
@echoix echoix added this to the 8.5.0 milestone Jul 1, 2024
@github-actions github-actions bot added vector Related to vector data processing raster Related to raster data processing Python Related code is in Python database Related to database management module general display imagery tests Related to Test Suite raster3d misc labels Jul 1, 2024
github-advanced-security[bot]
github-advanced-security bot found potential problems Jul 1, 2024
View reviewed changes
scripts/g.search.modules/g.search.modules.py
@@ -65,11 +65,10 @@

import os
import sys
import xml.etree.ElementTree as etree

Check notice

Code scanning / Bandit

Using xml.etree.ElementTree to parse untrusted XML data is known to be vulnerable to XML attacks. Replace xml.etree.ElementTree with the equivalent defusedxml package, or make sure defusedxml.defuse_stdlib() is called. Note

Using xml.etree.ElementTree to parse untrusted XML data is known to be vulnerable to XML attacks. Replace xml.etree.ElementTree with the equivalent defusedxml package, or make sure defusedxml.defuse_stdlib() is called.
scripts/r.in.wms/wms_cap_parsers.py
@@ -18,10 +18,9 @@
"""

import pathlib

import xml.etree.ElementTree as etree

Check notice

Code scanning / Bandit

Using xml.etree.ElementTree to parse untrusted XML data is known to be vulnerable to XML attacks. Replace xml.etree.ElementTree with the equivalent defusedxml package, or make sure defusedxml.defuse_stdlib() is called. Note

Using xml.etree.ElementTree to parse untrusted XML data is known to be vulnerable to XML attacks. Replace xml.etree.ElementTree with the equivalent defusedxml package, or make sure defusedxml.defuse_stdlib() is called.
scripts/v.import/v.import.py
import re # only needed for GDAL version < 2.4.1
import sys
import xml.etree.ElementTree as ET # only needed for GDAL version < 2.4.1

Check notice

Code scanning / Bandit

Using xml.etree.ElementTree to parse untrusted XML data is known to be vulnerable to XML attacks. Replace xml.etree.ElementTree with the equivalent defusedxml package, or make sure defusedxml.defuse_stdlib() is called. Note

Using xml.etree.ElementTree to parse untrusted XML data is known to be vulnerable to XML attacks. Replace xml.etree.ElementTree with the equivalent defusedxml package, or make sure defusedxml.defuse_stdlib() is called.
This was referenced Jul 1, 2024
Closed
Closed
Closed
@echoix echoix marked this pull request as draft July 1, 2024 22:27
@echoix echoix closed this Jul 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
database Related to database management display general imagery misc module Python Related code is in Python raster Related to raster data processing raster3d tests Related to Test Suite vector Related to vector data processing
1 participant
@echoix