Skip to content

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

Merged
MichaReiser merged 1 commit intoastral-sh:mainfrom
camper42:fix-airflow-help-msg
Apr 24, 2025
Merged

[airflow] fix typos (AIR302, AIR312)#17574
MichaReiser merged 1 commit intoastral-sh:mainfrom
camper42:fix-airflow-help-msg

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
@github-actions
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