Skip to content

Pbd/revert-project-root-refactoring#7446

Merged
petebacondarwin merged 5 commits intomainfrom
pbd/revert-project-root-refactoring
Dec 5, 2024
Merged

Pbd/revert-project-root-refactoring#7446
petebacondarwin merged 5 commits intomainfrom
pbd/revert-project-root-refactoring

Conversation

@petebacondarwin
Copy link
Contributor

Fixes #7440

Fix a regression introduced in #7415 with the tmp dir location for Pages projects
by reverting those changes.


  • Tests
    • TODO (before merge)
    • Tests included
    • Tests not necessary because:
  • E2E Tests CI Job required? (Use "e2e" label or ask maintainer to run separately)
    • I don't know
    • Required
    • Not required because:
  • Public documentation
    • TODO (before merge)
    • Cloudflare docs PR(s):
    • Documentation not necessary because: bug fix

@petebacondarwin petebacondarwin requested review from a team as code owners December 4, 2024 22:49
@changeset-bot
Copy link

changeset-bot bot commented Dec 4, 2024

🦋 Changeset detected

Latest commit: 425d1ee

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
wrangler Patch
@cloudflare/vitest-pool-workers Patch

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

@petebacondarwin petebacondarwin added the e2e Run wrangler + vite-plugin e2e tests on a PR label Dec 4, 2024
@petebacondarwin petebacondarwin force-pushed the pbd/revert-project-root-refactoring branch from 6debc6d to 425d1ee Compare December 4, 2024 22:50
@petebacondarwin petebacondarwin mentioned this pull request Dec 4, 2024
9 tasks
@github-actions
Copy link
Contributor

github-actions bot commented Dec 4, 2024

A wrangler prerelease is available for testing. You can install this latest build in your project with:

npm install --save-dev https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/12169900626/npm-package-wrangler-7446

You can reference the automatically updated head of this PR with:

npm install --save-dev https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/prs/7446/npm-package-wrangler-7446

Or you can use npx with this latest build directly:

npx https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/12169900626/npm-package-wrangler-7446 dev path/to/script.js
Additional artifacts:
npx https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/12169900626/npm-package-create-cloudflare-7446 --no-auto-update
npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/12169900626/npm-package-cloudflare-kv-asset-handler-7446
npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/12169900626/npm-package-miniflare-7446
npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/12169900626/npm-package-cloudflare-pages-shared-7446
npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/12169900626/npm-package-cloudflare-vitest-pool-workers-7446
npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/12169900626/npm-package-cloudflare-workers-editor-shared-7446
npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/12169900626/npm-package-cloudflare-workers-shared-7446
npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/12169900626/npm-package-cloudflare-workflows-shared-7446

Note that these links will no longer work once the GitHub Actions artifact expires.


wrangler@3.92.0 includes the following runtime dependencies:

Package Constraint Resolved
miniflare workspace:* 3.20241106.2
workerd 1.20241106.2 1.20241106.2
workerd --version 1.20241106.2 2024-11-06

Please ensure constraints are pinned, and miniflare/workerd minor versions match.

@petebacondarwin petebacondarwin force-pushed the pbd/revert-project-root-refactoring branch from 425d1ee to 7725249 Compare December 4, 2024 23:05
@petebacondarwin petebacondarwin merged commit 9435af0 into main Dec 5, 2024
@petebacondarwin petebacondarwin deleted the pbd/revert-project-root-refactoring branch December 5, 2024 11:26
@workers-devprod workers-devprod mentioned this pull request Dec 4, 2024
penalosa pushed a commit that referenced this pull request Jan 10, 2025
* fix up import-npm package lock

* Revert "refactor: remove missed redundant computation of `projectRoot` (#7421)"

This reverts commit bea6558.

* Revert "refactor: move projectRoot computation to config validation (#7415)"

This reverts commit 4f1a46e.

* Do not watch workflow tests in `test:ci` jobs

* add e2e test to prevent tmp directory regression
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

e2e Run wrangler + vite-plugin e2e tests on a PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

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

2 participants