Skip to content

Added static check for conn-fields defined in provider.yaml#69473

Open
dabla wants to merge 2 commits into
apache:mainfrom
dabla:feature/add-static-check-conn-fields-provider-yaml
Open

Added static check for conn-fields defined in provider.yaml#69473
dabla wants to merge 2 commits into
apache:mainfrom
dabla:feature/add-static-check-conn-fields-provider-yaml

Conversation

@dabla

@dabla dabla commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Added static check for conn-fields defined in provider.yaml to make sure those correspond with the ones defined in the get_connection_form_widgets method of the corresponding hook.


Was generative AI tooling used to co-author this PR?
  • [ x ] Yes (please specify the tool below)

Claude Sonnet 4.6


  • Read the Pull Request Guidelines for more information. Note: commit author/co-author name and email in commits become permanently public when merged.
  • For fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
  • When adding dependency, check compliance with the ASF 3rd Party License Policy.
  • For significant user-facing changes create newsfragment: {pr_number}.significant.rst, in airflow-core/newsfragments. You can add this file in a follow-up commit after the PR is created so you know the PR number.

@dabla dabla force-pushed the feature/add-static-check-conn-fields-provider-yaml branch from dcc3524 to 1f8b257 Compare July 6, 2026 13:43
@dabla dabla force-pushed the feature/add-static-check-conn-fields-provider-yaml branch 5 times, most recently from 70b85c3 to 92524b0 Compare July 6, 2026 22:20
… to make sure those correspond with the ones defined in the get_connection_form_widgets method of the corresponding hook
@dabla dabla force-pushed the feature/add-static-check-conn-fields-provider-yaml branch from 92524b0 to 0057094 Compare July 7, 2026 08:33
…t in sync with the fields defined in get_connection_form_widgets of their corresponding Hook
@dabla dabla requested a review from shahar1 as a code owner July 7, 2026 10:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant