Skip to content

fix: Push OpenaAPI generator client version choice down to the bazel client#64

Merged
BradHolmes merged 2 commits intoOpenAPITools:masterfrom
BradHolmes:correct-client-setup
Mar 24, 2025
Merged

fix: Push OpenaAPI generator client version choice down to the bazel client#64
BradHolmes merged 2 commits intoOpenAPITools:masterfrom
BradHolmes:correct-client-setup

Conversation

@BradHolmes
Copy link
Copy Markdown
Contributor

@BradHolmes BradHolmes commented Mar 1, 2025

Remove setting any sort of default client in the root bazel module. Require clients to register a client.

  • Done: Updated README

Tested:

  • Default client
  • Custom client version

@BradHolmes BradHolmes force-pushed the correct-client-setup branch 3 times, most recently from eb159b7 to 48f282b Compare March 1, 2025 19:14
@BradHolmes BradHolmes marked this pull request as ready for review March 2, 2025 23:10
@BradHolmes BradHolmes force-pushed the correct-client-setup branch from 48f282b to 1aa574d Compare March 6, 2025 14:31
@BradHolmes BradHolmes force-pushed the correct-client-setup branch from 3dbfbcf to 7e72aff Compare March 18, 2025 10:53
@BradHolmes BradHolmes requested a review from wing328 March 20, 2025 19:19
@BradHolmes BradHolmes self-assigned this Mar 20, 2025
@BradHolmes BradHolmes force-pushed the correct-client-setup branch from 7e72aff to 4d29dd4 Compare March 24, 2025 18:36
@BradHolmes BradHolmes merged commit e66d4f0 into OpenAPITools:master Mar 24, 2025
1 check passed
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