Add airflow-custom-provider skill - #257
Closed
odaneau-astro wants to merge 3 commits into
Closed
Conversation
Covers packaging a hook/operator/sensor integration as an installable Airflow provider: directory skeleton, the get_provider_info() contract and apache_airflow_provider entry point, constraints that silently break discovery or scheduler parsing, the wheel-build-and-astro-dev-start verification loop, and when a provider package is warranted versus a plain operator in a single DAG repo's include/ or plugins/. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Rewrites the skill against a fresh clone of astronomer/airflow-provider-sample to confirm the package skeleton, get_provider_info() required fields, and entry-point contract. Adds a decision table for provider package vs. a lighter include/plugins operator, a UI connection-form snippet, and af config providers/connections verification steps for the functional testing loop. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Fixes an inverted/garbled rule about __init__ constructors, corrects a cross-reference, and applies the Astronomer style guide (Latin abbreviations, "repository" not "repo", "folder" not "directory") plus a pass to remove AI-writing tells (em dashes, hollow intensifiers).
Author
|
Closing: opened against the wrong repo. This work is being redone against astronomer/hosted-skills. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
get_provider_info(), theapache_airflow_providerentry point, custom connection types).__init__constructors, a cross-reference, and applies the Astronomer style guide plus an AI-writing-tell cleanup pass (em dashes, Latin abbreviations, hollow intensifiers).Test plan
get_provider_info()/ entry-point verification commands against a real provider install