Skip to content

Add Stripe provider (apache-airflow-providers-stripe)#63401

Draft
andreahlert wants to merge 1 commit intoapache:mainfrom
andreahlert:feat/add-stripe-provider
Draft

Add Stripe provider (apache-airflow-providers-stripe)#63401
andreahlert wants to merge 1 commit intoapache:mainfrom
andreahlert:feat/add-stripe-provider

Conversation

@andreahlert
Copy link
Copy Markdown
Contributor

Introduces a new apache-airflow-providers-stripe provider for Apache Airflow, discussed on the dev mailing list at https://lists.apache.org/thread/r9fr6571w3ssx07bnd5lgc9m88r1xy3c

What's included:

  • StripeHook backed by the official stripe-python SDK (>=7.0.0)
  • Automatic pagination via auto_paging_iter() — no memory blow-up on large result sets
  • list_all() and retrieve() convenience methods
  • stripe connection type with UI field behaviour (API key in the Password field)
  • Optional Stripe API version pinning via connection Extra field
  • 12 unit tests covering caching, pagination, error paths, and the four main resources (Customer, Subscription, Invoice, Event)
  • RST documentation and connection guide
  • lifecycle: incubation per AIP-95

Was generative AI tooling used to co-author this PR?
  • Yes

@andreahlert
Copy link
Copy Markdown
Contributor Author

The failing checks are unrelated to this PR:

  • airflow-ctl:P3.11/P3.13 and Test git clone on Windows: transient GitHub Actions infrastructure error (fatal: could not read Username for 'https://github.com': terminal prompts disabled) — runner failed to checkout the fork. Same error seen on other PRs today.

  • Breeze unit tests: failing on test_validate_provider_info_with_schema for the OpenAI provider ('ai-inference' is not of type 'object'), pre-existing on main. Also visible on Improve auto-triage: fake-SUCCESS detection, check status display, and triage tracking #63389.

All provider-specific checks pass. Locally verified: 12/12 unit tests pass, prek run --from-ref main --stage pre-commit clean.

@potiuk
Copy link
Copy Markdown
Member

potiuk commented Mar 12, 2026

You need to rebase. Also Nobody will review the PR if it is "Draft" - see the guidelines.

@andreahlert andreahlert force-pushed the feat/add-stripe-provider branch from 18fd5d9 to 2e6261b Compare March 12, 2026 09:41
@andreahlert andreahlert marked this pull request as ready for review March 12, 2026 09:41
@andreahlert andreahlert force-pushed the feat/add-stripe-provider branch from 2e6261b to 461898d Compare March 12, 2026 11:28
@choo121600 choo121600 marked this pull request as draft March 13, 2026 03:20
@choo121600
Copy link
Copy Markdown
Member

@andreahlert This PR has been converted to draft because it does not yet meet our Pull Request quality criteria.

Issues found:

  • Merge conflicts: This PR has merge conflicts with the main branch. Your branch is 33 commits behind main. Please rebase your branch (git fetch origin && git rebase origin/main), resolve the conflicts, and push again. See contributing quick start.
  • Pre-commit / static checks: Failing: CI image checks / Static checks. Run prek run --from-ref main locally to find and fix issues. See Pre-commit / static checks docs.
  • Build docs: Failing: CI image checks / Build documentation (--docs-only). Run breeze build-docs locally to reproduce. See Build docs docs.
  • Other failing CI checks: Failing: CI image checks / Build documentation (--spellcheck-only). Run prek run --from-ref main locally to reproduce. See static checks docs.

Note: Your branch is 33 commits behind main. Some check failures may be caused by changes in the base branch rather than by your PR. Please rebase your branch and push again to get up-to-date CI results.

What to do next:

  • The comment informs you what you need to do.
  • Fix each issue, then mark the PR as "Ready for review" in the GitHub UI - but only after making sure that all the issues are fixed.
  • Maintainers will then proceed with a normal review.

Converting a PR to draft is not a rejection — it is an invitation to bring the PR up to the project's standards so that maintainer review time is spent productively. If you have questions, feel free to ask on the Airflow Slack.

@andreahlert andreahlert force-pushed the feat/add-stripe-provider branch 3 times, most recently from e4105cc to 8153cb1 Compare March 16, 2026 08:29
@potiuk potiuk closed this Apr 6, 2026
@potiuk potiuk added the closed because of multiple quality violations Label used to close the PRs when there are multiple quality violations label Apr 6, 2026
@potiuk
Copy link
Copy Markdown
Member

potiuk commented Apr 6, 2026

This draft pull request has had no activity from the author for over 3 weeks.

@andreahlert, you are welcome to reopen this PR when you are ready to continue working on it. Thank you for your contribution!

@jscheffl
Copy link
Copy Markdown
Contributor

jscheffl commented Apr 6, 2026

@potiuk I assume this is because of ongoing discussions on devlist?

@potiuk potiuk reopened this Apr 8, 2026
@kaxil kaxil requested a review from Copilot April 10, 2026 19:55
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@andreahlert andreahlert force-pushed the feat/add-stripe-provider branch from 8153cb1 to bb26c95 Compare May 5, 2026 12:11
Introduces a new Stripe provider for Apache Airflow, discussed on the
dev mailing list at https://lists.apache.org/thread/r9fr6571w3ssx07bnd5lgc9m88r1xy3c

Changes included:
- StripeHook backed by the official stripe-python SDK (>=8.0.0)
- Automatic pagination via auto_paging_iter()
- list_all() and retrieve() convenience methods
- stripe connection type with UI field behaviour (API key in Password field)
- Optional Stripe API version pinning via connection Extra
- 12 unit tests covering caching, pagination, error paths, and all four main resources
- RST documentation and connection guide
- lifecycle: incubation per AIP-95
- Registered in workspace pyproject.toml, boring-cyborg.yml, extra-packages-ref, bug report template

Signed-off-by: André Ahlert <andre@aex.partners>
@andreahlert andreahlert force-pushed the feat/add-stripe-provider branch from bb26c95 to 35ef691 Compare May 5, 2026 12:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:dev-tools area:providers closed because of multiple quality violations Label used to close the PRs when there are multiple quality violations kind:documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants