Skip to content

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

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

Merged
merged 2 commits into from
Mar 24, 2025

Conversation

BradHolmes
Copy link
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.

1 participant