Skip to content

Conversation

jar-stripe
Copy link
Contributor

@jar-stripe jar-stripe commented Sep 19, 2025

Why?

just format is run by our code generator after generating code. I noticed it was only formatting code in Stripe.net and not in StripeTests (which contains generated content). This PR changes just format to use the .sln file which should cover all projects in the repo.

Is it slower?

Marginally, but I think its ok.

Before:

159.90s user 6.98s system 325% cpu 51.298 total

After:

163.53s user 7.66s system 296% cpu 57.724 total

What?

  • changes format recipe in justfile to use the .sln file
  • changes format in generated StripeTests files and in Example files

@jar-stripe jar-stripe requested a review from a team as a code owner September 19, 2025 19:28
@jar-stripe jar-stripe requested review from prathmesh-stripe and removed request for a team September 19, 2025 19:28
@jar-stripe jar-stripe enabled auto-merge (squash) September 19, 2025 19:30
@jar-stripe jar-stripe requested review from xavdid-stripe and removed request for prathmesh-stripe September 19, 2025 19:30
Copy link
Member

@xavdid-stripe xavdid-stripe left a comment

Choose a reason for hiding this comment

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

Is it slower?

😭😭😭

Like you're right, it's fine, but it's just so slow already 🙈

@jar-stripe jar-stripe merged commit 6fb0d45 into master Sep 19, 2025
4 of 5 checks passed
@jar-stripe jar-stripe deleted the jar/format-entire-solution branch September 19, 2025 20:30
helenye-stripe pushed a commit that referenced this pull request Sep 23, 2025
* Add private-preview branch to CI workflow triggers

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>

* ⚠️ Build SDK w/ V2 OpenAPI spec (#3167)

* generate w/ v2 openapi spec

* move deleted object path

* Adds public BaseUrl to RawRequestOptions (#3170)

renames internal BaseUrl in RequestOptions to InternalBaseUrl 
adds public BaseUrl to RawRequestOptions - it delegates to InternalBaseUrl for safety

* Update generated code (#3172)

* Update generated code for v1955 and 2025-09-30.clover

* Update generated code for v1968 and 2025-09-30.clover

---------

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

* `just format` formats entire solution (#3174)

* Adds ability to specify file name and type when calling FileService.Create (#3171)

* Update generated code for 9de7288a5c444f47d15545549303c3de4c226c71 and 2025-09-30.clover

---------

Co-authored-by: Michael Broshi <[email protected]>
Co-authored-by: Claude <[email protected]>
Co-authored-by: David Brownman <[email protected]>
Co-authored-by: jar-stripe <[email protected]>
Co-authored-by: Stripe OpenAPI <105521251+stripe-openapi[bot]@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