Skip to content

feat(#2160): support @tanstack/vue-query #2161

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

Closed
wants to merge 1 commit into from

Conversation

hirotaka
Copy link

@hirotaka hirotaka commented Feb 18, 2025

Changes

Implements #2160.

How to Review

How can a reviewer review your changes? What should be kept in mind for this review?

Checklist

  • Unit tests updated
  • docs/ updated (if necessary)
  • pnpm run update:examples run (only applicable for openapi-typescript)

Sorry, something went wrong.

@hirotaka hirotaka requested a review from a team as a code owner February 18, 2025 11:00
@hirotaka hirotaka requested a review from gzm0 February 18, 2025 11:00
Copy link

netlify bot commented Feb 18, 2025

👷 Deploy request for openapi-ts pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit ed1a7e6

Copy link

changeset-bot bot commented Feb 18, 2025

⚠️ No Changeset found

Latest commit: ed1a7e6

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@hirotaka hirotaka marked this pull request as draft February 18, 2025 11:00
@hirotaka hirotaka force-pushed the feat-vue-query-support branch 2 times, most recently from 70f0490 to 51764e8 Compare March 6, 2025 14:36
@hirotaka hirotaka force-pushed the feat-vue-query-support branch from 51764e8 to ed1a7e6 Compare March 6, 2025 14:37
@drwpow
Copy link
Contributor

drwpow commented May 3, 2025

@hirotaka thanks so much for your proposal, and for doing all the work! Unfortunately we can’t accept this PR because of some shifts in ownership structure (it’s not you, it’s us!).

Over the past several months we’ve been shifting the project into less me-led (the original creator), and more community-led through governance and shared ownership. We’ve had some discussions about project direction more specifically last few weeks, and have come to the conclusion that we can’t maintain all the ecosystem wrappers for openapi-fetch. In fact, in the future, we may even have to deprecate the openapi-react-query/swr-openapi wrappers. The only thing we know is we can’t take on additional packages.

This actually conflicts with things we’ve said we’d accept in the past, so this is a shift which is on us! But we’re trying to offshed maintenance so that the core can be stronger, and we can go deeper on a few things rather than going too shallow/too little on too many things.


So all that to say, I would love to help set you up to publish/own this yourself! Feel free to “steal” any infrastructure or setup here (it’s MIT licensed!), and please tag me on your own repo and I would be happy to help you publish/maintain this yourself. I would love for this to exist, especially since you did all the work! Just at this time we don’t have the resources to maintain it, and we would hamper its development.

Please let me know how I can help you push this forward outside of this repo though!

@drwpow drwpow closed this May 3, 2025
@drwpow drwpow mentioned this pull request May 3, 2025
1 task
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.

None yet

2 participants