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

remove test using soon-to-be-gone endpoint #2270

Merged
merged 2 commits into from
Mar 13, 2025
Merged

Conversation

xavdid-stripe
Copy link
Member

Why?

The /v1/invoices/upcoming/lines endpoint is being removed soon. As a result, this test is causing CI failures upstream. I'm removing it for now in the hopes that we can replace it with a endpoint that accomplishes the same thing, but still exists.

Another test was removed entirely.

We don't need to change anything auto-generated; that'll get picked up when the spec is updated. These changes are purely for manual files.

What?

  • comment out test
  • remove endpoint-specific test

See Also

@xavdid-stripe xavdid-stripe requested a review from a team as a code owner March 13, 2025 00:31
@xavdid-stripe xavdid-stripe requested review from mbroshi-stripe and removed request for a team March 13, 2025 00:31
@xavdid-stripe xavdid-stripe enabled auto-merge (squash) March 13, 2025 00:31
@xavdid-stripe xavdid-stripe merged commit 6c78dcb into master Mar 13, 2025
8 checks passed
@xavdid-stripe xavdid-stripe deleted the remove-upcoming-lines branch March 13, 2025 14:14
ramya-stripe added a commit that referenced this pull request Mar 13, 2025
* Add Next.js App Router webhook example that leverages Route Handlers (#2259)

* Update generated code (#2256)

* Update generated code for v1463

* Update generated code for v1494

* Update generated code for v1495

* Update generated code for v1501

* Update generated code for v1505

---------

Co-authored-by: Stripe OpenAPI <105521251+stripe-openapi[bot]@users.noreply.github.com>
Co-authored-by: prathmesh-stripe <165320323+prathmesh-stripe@users.noreply.github.com>

* Bump version to 17.7.0

* Update actions/cache as v2 is deprecated (#2269)

* remove test using soon-to-be-gone endpoint (#2270)

---------

Co-authored-by: Jonathan Steele <83410553+jsteele-stripe@users.noreply.github.com>
Co-authored-by: stripe-openapi[bot] <105521251+stripe-openapi[bot]@users.noreply.github.com>
Co-authored-by: prathmesh-stripe <165320323+prathmesh-stripe@users.noreply.github.com>
Co-authored-by: Prathmesh Ranaut <prathmesh@stripe.com>
Co-authored-by: David Brownman <109395161+xavdid-stripe@users.noreply.github.com>
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.

2 participants