Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Check path string properly #9367

Merged
merged 1 commit into from
Jan 2, 2024
Merged

Conversation

nickdrozd
Copy link
Contributor

A minor whoopsie, 158367b forgot to update this line.

I'm not sure how this gets tested in CI.

A minor whoopsie, 158367b forgot to update this line.
@nickdrozd
Copy link
Contributor Author

Fixes #9366

@charliermarsh
Copy link
Member

We do test the python -m thing as part of the release process, but perhaps not on all platforms.

Copy link
Member

@charliermarsh charliermarsh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you! I'll look into the test coverage as well.

@charliermarsh charliermarsh added the bug Something isn't working label Jan 2, 2024
@charliermarsh charliermarsh enabled auto-merge (squash) January 2, 2024 21:01
@charliermarsh charliermarsh merged commit 08c60f5 into astral-sh:main Jan 2, 2024
15 of 16 checks passed
@nickdrozd nickdrozd deleted the path-type-error branch January 2, 2024 21:05
Copy link
Contributor

github-actions bot commented Jan 2, 2024

ruff-ecosystem results

Linter (stable)

ℹ️ ecosystem check detected linter changes. (+1653 -1653 violations, +0 -0 fixes in 2 projects; 39 projects unchanged)

apache/airflow (+42 -42 violations, +0 -0 fixes)

ruff check --no-cache --exit-zero --no-preview --select ALL

+ tests/providers/amazon/aws/hooks/test_eks.py:57:1: TID252 Prefer absolute imports over relative imports from parent modules
+ tests/providers/amazon/aws/hooks/test_eks.py:57:1: TID252 Prefer absolute imports over relative imports from parent modules
+ tests/providers/amazon/aws/hooks/test_eks.py:57:1: TID252 Prefer absolute imports over relative imports from parent modules
+ tests/providers/amazon/aws/hooks/test_eks.py:57:1: TID252 Prefer absolute imports over relative imports from parent modules
+ tests/providers/amazon/aws/hooks/test_eks.py:57:1: TID252 Prefer absolute imports over relative imports from parent modules
+ tests/providers/amazon/aws/hooks/test_eks.py:57:1: TID252 Prefer absolute imports over relative imports from parent modules
+ tests/providers/amazon/aws/hooks/test_eks.py:57:1: TID252 Prefer absolute imports over relative imports from parent modules
+ tests/providers/amazon/aws/hooks/test_eks.py:57:1: TID252 Prefer absolute imports over relative imports from parent modules
+ tests/providers/amazon/aws/hooks/test_eks.py:57:1: TID252 Prefer absolute imports over relative imports from parent modules
+ tests/providers/amazon/aws/hooks/test_eks.py:57:1: TID252 Prefer absolute imports over relative imports from parent modules
... 74 additional changes omitted for project

bokeh/bokeh (+1611 -1611 violations, +0 -0 fixes)

ruff check --no-cache --exit-zero --no-preview --select ALL

+ src/bokeh/application/application.py:42:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/application.py:42:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/application.py:43:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/application.py:43:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/application.py:44:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/application.py:44:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/application.py:50:5: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/application.py:50:5: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/code.py:45:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/code.py:45:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/code.py:46:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/code.py:46:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/code.py:47:1: TID252 Prefer absolute imports over relative imports from parent modules
+ src/bokeh/application/handlers/code.py:47:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/code.py:47:1: TID252 Relative imports from parent modules are banned
- src/bokeh/application/handlers/code.py:47:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/code_runner.py:33:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/code_runner.py:33:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/code_runner.py:34:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/code_runner.py:34:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/directory.py:65:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/directory.py:65:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/directory.py:66:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/directory.py:66:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/directory.py:67:1: TID252 Prefer absolute imports over relative imports from parent modules
+ src/bokeh/application/handlers/directory.py:67:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/directory.py:67:1: TID252 Relative imports from parent modules are banned
- src/bokeh/application/handlers/directory.py:67:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/directory.py:78:5: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/directory.py:78:5: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/document_lifecycle.py:25:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/document_lifecycle.py:25:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/function.py:44:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/function.py:44:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/function.py:45:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/function.py:45:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/handler.py:52:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/handler.py:52:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/handler.py:53:1: TID252 Prefer absolute imports over relative imports from parent modules
+ src/bokeh/application/handlers/handler.py:53:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/handler.py:53:1: TID252 Relative imports from parent modules are banned
- src/bokeh/application/handlers/handler.py:53:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/lifecycle.py:28:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/lifecycle.py:28:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/lifecycle.py:29:1: TID252 Prefer absolute imports over relative imports from parent modules
+ src/bokeh/application/handlers/lifecycle.py:29:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/lifecycle.py:29:1: TID252 Relative imports from parent modules are banned
- src/bokeh/application/handlers/lifecycle.py:29:1: TID252 Relative imports from parent modules are banned
... 3174 additional changes omitted for project

Changes by rule (1 rules affected)

code total + violation - violation + fix - fix
TID252 3306 1653 1653 0 0

Linter (preview)

ℹ️ ecosystem check detected linter changes. (+1572 -1572 violations, +0 -0 fixes in 2 projects; 39 projects unchanged)

apache/airflow (+42 -42 violations, +0 -0 fixes)

ruff check --no-cache --exit-zero --preview --select ALL

+ tests/providers/amazon/aws/hooks/test_eks.py:57:1: TID252 Prefer absolute imports over relative imports from parent modules
+ tests/providers/amazon/aws/hooks/test_eks.py:57:1: TID252 Prefer absolute imports over relative imports from parent modules
+ tests/providers/amazon/aws/hooks/test_eks.py:57:1: TID252 Prefer absolute imports over relative imports from parent modules
+ tests/providers/amazon/aws/hooks/test_eks.py:57:1: TID252 Prefer absolute imports over relative imports from parent modules
+ tests/providers/amazon/aws/hooks/test_eks.py:57:1: TID252 Prefer absolute imports over relative imports from parent modules
+ tests/providers/amazon/aws/hooks/test_eks.py:57:1: TID252 Prefer absolute imports over relative imports from parent modules
+ tests/providers/amazon/aws/hooks/test_eks.py:57:1: TID252 Prefer absolute imports over relative imports from parent modules
+ tests/providers/amazon/aws/hooks/test_eks.py:57:1: TID252 Prefer absolute imports over relative imports from parent modules
+ tests/providers/amazon/aws/hooks/test_eks.py:57:1: TID252 Prefer absolute imports over relative imports from parent modules
+ tests/providers/amazon/aws/hooks/test_eks.py:57:1: TID252 Prefer absolute imports over relative imports from parent modules
... 74 additional changes omitted for project

bokeh/bokeh (+1530 -1530 violations, +0 -0 fixes)

ruff check --no-cache --exit-zero --preview --select ALL

+ src/bokeh/application/handlers/code.py:45:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/code.py:45:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/code.py:46:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/code.py:46:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/code.py:47:1: TID252 Prefer absolute imports over relative imports from parent modules
+ src/bokeh/application/handlers/code.py:47:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/code.py:47:1: TID252 Relative imports from parent modules are banned
- src/bokeh/application/handlers/code.py:47:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/code_runner.py:33:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/code_runner.py:33:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/code_runner.py:34:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/code_runner.py:34:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/directory.py:65:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/directory.py:65:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/directory.py:66:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/directory.py:66:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/directory.py:67:1: TID252 Prefer absolute imports over relative imports from parent modules
+ src/bokeh/application/handlers/directory.py:67:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/directory.py:67:1: TID252 Relative imports from parent modules are banned
- src/bokeh/application/handlers/directory.py:67:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/directory.py:78:5: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/directory.py:78:5: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/document_lifecycle.py:25:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/document_lifecycle.py:25:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/function.py:44:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/function.py:44:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/function.py:45:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/function.py:45:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/handler.py:52:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/handler.py:52:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/handler.py:53:1: TID252 Prefer absolute imports over relative imports from parent modules
+ src/bokeh/application/handlers/handler.py:53:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/handler.py:53:1: TID252 Relative imports from parent modules are banned
- src/bokeh/application/handlers/handler.py:53:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/lifecycle.py:28:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/lifecycle.py:28:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/lifecycle.py:29:1: TID252 Prefer absolute imports over relative imports from parent modules
+ src/bokeh/application/handlers/lifecycle.py:29:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/lifecycle.py:29:1: TID252 Relative imports from parent modules are banned
- src/bokeh/application/handlers/lifecycle.py:29:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/notebook.py:36:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/notebook.py:36:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/notebook.py:37:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/notebook.py:37:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/script.py:52:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/script.py:52:1: TID252 Relative imports from parent modules are banned
+ src/bokeh/application/handlers/server_lifecycle.py:29:1: TID252 Prefer absolute imports over relative imports from parent modules
- src/bokeh/application/handlers/server_lifecycle.py:29:1: TID252 Relative imports from parent modules are banned
... 3012 additional changes omitted for project

Changes by rule (1 rules affected)

code total + violation - violation + fix - fix
TID252 3144 1572 1572 0 0

Formatter (stable)

✅ ecosystem check detected no format changes.

Formatter (preview)

✅ ecosystem check detected no format changes.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants