-
Notifications
You must be signed in to change notification settings - Fork 18
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
feat: hono party #157
feat: hono party #157
Conversation
🦋 Changeset detectedLatest commit: f51bf60 The changes in this PR will be included in the next version bump. This PR includes changesets to release 6 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
7446b07
to
08b68bb
Compare
08b68bb
to
0411b0e
Compare
@yusukebe should this be in this repo or the hono repo? whatever you prefer, happy to do that. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good to me! Approving for now, and we'll decide what to do with it. Would you like to make a small example under the fixtures folder? No worries if not
packages/hono-party/package.json
Outdated
"hono": "^4", | ||
"partyserver": "workspace:*" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe these should be peer deps too
packages/hono-party/package.json
Outdated
"license": "ISC", | ||
"description": "", | ||
"dependencies": { | ||
"hono": "^4", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hono
also should be peerDependencies
.
We can include this in our |
This is great! What I was wondering was whether to include |
taking over this pr, will try to land it today |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! I’m going to pub
this implements a middleware for hono so you can expose many partyservers from a single hono app :)