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

[dagster-airlift][doc] Remove outdated references BaseProxyToDagsterOperator in README #25347

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

christeefy
Copy link
Contributor

Summary & Motivation

In dagster-airlift, the class BaseProxyToDagsterOperator was renamed to BaseProxyTaskToDagsterOperator in v0.0.26.

In the public-facing Dagster Airlift tutorial, there are still some references to this outdated class name in the README.

This PR:

  • updates mentions of BaseProxyToDagsterOperator in the README
  • updates the corresponding Airflow DAG screenshot containing the BaseProxyToDagsterOperator
  • updates the Airflow DAG's proxied state tag to be in line with the tag created by the dagster-airlift implementation (v0.0.26)

How I Tested These Changes

  • Verified that there are no more references to BaseProxyToDagsterOperator via a repo-level keyword search
  • Verified all images have no references to BaseProxyToDagsterOperator

Changelog

NOCHANGELOG

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.

1 participant