Skip to content

[airflow] fix typos (AIR302, AIR312) #17574

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

Merged
merged 1 commit into from
Apr 24, 2025

Conversation

camper42
Copy link
Contributor

@camper42 camper42 commented Apr 23, 2025

Summary

Fix Airflow Provider package name

apache-airflow-provider-{provider} -> apache-airflow-providers-{provider} (provider -> providers)

with ruff 0.11.6, package name in help msg is not correct, e.g.:

   = help: Install `apache-airflow-provider-standard>=0.0.1` and use `airflow.providers.standard.operators.python.ShortCircuitOperator` instead.

when running uv add apache-airflow-provider-standard, I get:

❯ uv add apache-airflow-provider-standard
  × No solution found when resolving dependencies for split (python_full_version >= '3.13'):
  ╰─▶ Because apache-airflow-provider-standard was not found in the package registry and your project depends on apache-airflow-provider-standard, we can conclude that your project's requirements
      are unsatisfiable.
  help: If you want to add the package regardless of the failed resolution, provide the `--frozen` flag to skip locking and syncing.

correct package name is apache-airflow-providers-standard

ref: https://airflow.apache.org/docs/apache-airflow-providers-standard/stable/index.html

Test Plan

The test fixture has been updated accordingly

`apache-airflow-provider-{provider}` -> `apache-airflow-providers-{provider}`
Copy link
Contributor

@ntBre ntBre left a comment

Choose a reason for hiding this comment

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

Thanks! @Lee-W does this look good to you?

@ntBre ntBre added the bug Something isn't working label Apr 23, 2025
Copy link
Contributor

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

@Lee-W
Copy link
Contributor

Lee-W commented Apr 24, 2025

Yep, thanks for pointing out!

@MichaReiser MichaReiser merged commit 48a85c4 into astral-sh:main Apr 24, 2025
33 checks passed
dcreager added a commit that referenced this pull request Apr 24, 2025
* main:
  [red-knot] fix collapsing literal and its negation to object (#17605)
  [red-knot] Add more tests for protocols (#17603)
  [red-knot] Ban direct instantiations of `Protocol` classes (#17597)
  [`pyupgrade`] Preserve parenthesis when fixing native literals containing newlines (`UP018`) (#17220)
  [`airflow`] fix typos (`AIR302`, `AIR312`) (#17574)
  [red-knot] Special case `@abstractmethod` for function type (#17591)
  [red-knot] Emit diagnostics for isinstance() and issubclass() calls where a non-runtime-checkable protocol is the second argument (#17561)
  [red-knot] Infer the members of a protocol class (#17556)
  [red-knot] Add `FunctionType::to_overloaded` (#17585)
  [red-knot] Add mdtests for `global` statement (#17563)
  [syntax-errors] Make duplicate parameter names a semantic error (#17131)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy