Skip to content

custom billing plan skippable messages#387

Merged
mattmueller-stripe merged 1 commit into
mainfrom
skippable-billing
Jun 3, 2026
Merged

custom billing plan skippable messages#387
mattmueller-stripe merged 1 commit into
mainfrom
skippable-billing

Conversation

@mattmueller-stripe
Copy link
Copy Markdown

Summary

  • Adds a few more skippable messages for custom billing plans

How to test (optional)

Related

  • Closes #

Thanks for contributing ❤️

Copilot AI review requested due to automatic review settings June 3, 2026 17:50
@mattmueller-stripe mattmueller-stripe changed the title more skippable messages custom billing plan skippable messages Jun 3, 2026
Copy link
Copy Markdown

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.

Pull request overview

Adds additional Stripe “custom billing plan” error message patterns to the list of known skippable list errors, so affected streams are marked as skip (instead of failing) when these permanent 400s occur.

Changes:

  • Add two new skippable Stripe error-message substrings for billing_alerts and billing_credit_grants custom-plan limitations.
  • Extend the existing parameterized test coverage to assert stream_status: skip is emitted for these errors.
  • Minor whitespace-only cleanup on an existing concurrency comment line.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
packages/source-stripe/src/src-list-api.ts Adds new skippable error-message patterns for custom-plan billing endpoints; no functional changes elsewhere beyond message matching.
packages/source-stripe/src/index.test.ts Adds new test cases ensuring these custom-plan errors produce stream_status with skip.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@mattmueller-stripe mattmueller-stripe enabled auto-merge (squash) June 3, 2026 18:00
@mattmueller-stripe mattmueller-stripe merged commit 53b893f into main Jun 3, 2026
20 checks passed
@mattmueller-stripe mattmueller-stripe deleted the skippable-billing branch June 3, 2026 18:02
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.

3 participants