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
pages-shared 'wrangler pages dev' #1788
Conversation
🦋 Changeset detectedLatest commit: 3ef723b The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
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 |
82d5529
to
187ca14
Compare
A wrangler prerelease is available for testing. You can install this latest build in your project with: npm install --save-dev https://prerelease-registry.developers.workers.dev/runs/3022676615/npm-package-wrangler-1788 You can reference the automatically updated head of this PR with: npm install --save-dev https://prerelease-registry.developers.workers.dev/prs/1788/npm-package-wrangler-1788 Or you can use npx https://prerelease-registry.developers.workers.dev/runs/3022676615/npm-package-wrangler-1788 dev path/to/script.js |
48f8c98
to
ab31b9b
Compare
ab31b9b
to
d244090
Compare
d244090
to
ffd3266
Compare
Codecov Report
@@ Coverage Diff @@
## main #1788 +/- ##
==========================================
+ Coverage 75.36% 75.51% +0.14%
==========================================
Files 97 98 +1
Lines 7076 7160 +84
Branches 1849 1882 +33
==========================================
+ Hits 5333 5407 +74
- Misses 1743 1753 +10
|
if ( | ||
contentRequest.cf && | ||
"clientAcceptEncoding" in contentRequest.cf && | ||
contentRequest.cf.clientAcceptEncoding | ||
) { | ||
rawAcceptEncoding = contentRequest.cf.clientAcceptEncoding as string; |
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.
Am I right in thinking that this may resolve #370 if this code is also used in production?
(perhaps #165 (comment) too?)
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.
Not quite. We need to also assume a default "identity" accept-encoding if coming from env.ASSETS.fetch()
, but that is something we'll be able to tackle shortly :)
I think it will probably help that comment on 165 though :)
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.
Progress!
1850af6
to
1049e36
Compare
1049e36
to
3ef723b
Compare
Fixes #1001
Blocked bycloudflare/miniflare#358#1795Ready to go! 🎉