Skip to content

Revert "support backend hmr" - #302

Merged
benjreinhart merged 1 commit into
mainfrom
revert-277-bhmr
Sep 16, 2024
Merged

benjreinhart merged 1 commit into
mainfrom
revert-277-bhmr

Conversation

@benjreinhart

Copy link
Copy Markdown
Contributor

Reverts #277

@swk777 after merge, i ran pnpm dev and it spins up the API server twice, the second one failing due to port in use. I'm going to revert until that's fixed. My apologies, I should have verified this before merging.

@benjreinhart
benjreinhart merged commit 5c3b74d into main Sep 16, 2024
@benjreinhart
benjreinhart deleted the revert-277-bhmr branch September 16, 2024 17:43
@swk777

swk777 commented Sep 18, 2024

Copy link
Copy Markdown
Contributor

Reverts #277

@swk777 after merge, i ran pnpm dev and it spins up the API server twice, the second one failing due to port in use. I'm going to revert until that's fixed. My apologies, I should have verified this before merging.

No worries! What’s your environment? I tried it on my side, and it seems to be working fine.

@benjreinhart

Copy link
Copy Markdown
Contributor Author

Interesting.. I'm on mac OS 14.5 using iTerm. I can try again later but at the time I tried I gave it a few goes and rebuilt/redownloaded deps and it was still happening

@swk777

swk777 commented Sep 19, 2024

Copy link
Copy Markdown
Contributor

Interesting.. I'm on mac OS 14.5 using iTerm. I can try again later but at the time I tried I gave it a few goes and rebuilt/redownloaded deps and it was still happening

It might be related to the dependencies between shared and api. I will keep working on it.

@swk777 swk777 mentioned this pull request Sep 29, 2024
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