Skip to content
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

🐛 BUG: Latest wrangler causing a ton of hot reloads because it's putting the .wrangler/tmp directory inside /functions #7440

Closed
treeder opened this issue Dec 4, 2024 · 6 comments · Fixed by #7446
Labels
bug Something that isn't working regression Break in existing functionality as a result of a recent change

Comments

@treeder
Copy link

treeder commented Dec 4, 2024

Which Cloudflare product(s) does this pertain to?

Wrangler

What version(s) of the tool(s) are you using?

3.92.0

What version of Node are you using?

20.17.0

What operating system and version are you using?

Debian

Describe the Bug

Observed behavior

Doing a bunch of work in /functions/.wrangler directory.

image

Screen.recording.2024-12-04.9.12.01.AM.webm

Expected behavior

It should be doing it in /.wrangler like previous versions.

Steps to reproduce

npx wrangler pages dev ./public --d1 D1=dev

Then edit and save a file. In the terminal you'll see a bunch of things like this:

image

Please provide a link to a minimal reproduction

No response

Please provide any relevant error logs

No response

@treeder treeder added the bug Something that isn't working label Dec 4, 2024
@github-project-automation github-project-automation bot moved this to Untriaged in workers-sdk Dec 4, 2024
@penalosa penalosa added the regression Break in existing functionality as a result of a recent change label Dec 4, 2024
@jlobos
Copy link

jlobos commented Dec 4, 2024

some bug in new version 3.92.0 regression to 3.91.0 and normal hot reloading

@petebacondarwin petebacondarwin moved this from Untriaged to In Progress in workers-sdk Dec 4, 2024
@petebacondarwin
Copy link
Contributor

Sorry about this. The regression came in #7415

@petebacondarwin
Copy link
Contributor

Should go out in today's release

@treeder
Copy link
Author

treeder commented Dec 5, 2024

Any idea when the release will go out?

@petebacondarwin
Copy link
Contributor

We had failing e2e tests in the release PR, which we are working through, and the release should go out this morning once we have resolved those.

@treeder
Copy link
Author

treeder commented Dec 6, 2024

Surprised this release wasn't rolled back immediately. 3 days to leave a really busted release out in the wild is not a good look.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something that isn't working regression Break in existing functionality as a result of a recent change
Projects
None yet
4 participants