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

Make an exception with _3d import on QGS101 and QGS102 #11

Merged
merged 1 commit into from
Apr 19, 2024
Merged

Conversation

LKajan
Copy link
Contributor

@LKajan LKajan commented Apr 19, 2024

Should be merged after #10.

Usually QGS101 or QGS102 is thrown when importing _ prefixed modules.
_3d is currently an exception to this rule, so we should not throw an
error when importing it. Importing just 3d would throw and SyntaxError
because 3d is not a valid package name because of the leading digit.
_3d is the only way to use the 3d module.

Resolves: #9

@LKajan LKajan requested review from nmaarnio and ismogis April 19, 2024 11:53
@LKajan LKajan self-assigned this Apr 19, 2024
Usually QGS101 or QGS102 is thrown when importing _ prefixed modules.
_3d is currently an exception to this rule, so we should not throw an
error when importing it. Importing just 3d would throw and SyntaxError
because 3d is not a valid package name because of the leading digit.
_3d is the only way to use the 3d module.
@ismogis ismogis merged commit 66c1a3a into main Apr 19, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Imports from qgis._3d are not allowed.
2 participants