-
Notifications
You must be signed in to change notification settings - Fork 742
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
[C3] Address e2e flakiness #3877
Conversation
|
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/6087734672/npm-package-wrangler-3877 You can reference the automatically updated head of this PR with: npm install --save-dev https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/prs/6087734672/npm-package-wrangler-3877 Or you can use npx https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/6087734672/npm-package-wrangler-3877 dev path/to/script.js Additional artifacts:npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/6087734672/npm-package-cloudflare-pages-shared-3877 Note that these links will no longer work once the GitHub Actions artifact expires.
Please ensure constraints are pinned, and |
Codecov Report
@@ Coverage Diff @@
## main #3877 +/- ##
==========================================
- Coverage 75.25% 75.12% -0.13%
==========================================
Files 195 195
Lines 11452 11482 +30
Branches 3022 3033 +11
==========================================
+ Hits 8618 8626 +8
- Misses 2834 2856 +22 |
005de58
to
8855aad
Compare
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.
nice set of updates!
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.
Is it only Astro v4 that needs node.js 18? If so, then could we avoid bumping that framework in this PR, which would allow us to avoid the breaking change and major version bump.
Then we can bump Astro and, if necessary, the node.js version in a later PR.
Unfortunately it's not just Astro v4. I discovered this on an e2e run before I made this PR because the existing version of Astro we're using no longer works with node v16. I think when they released these changes they failed to pin This can be confirmed by running This kind of puts us in a pickle because to get the tests to work, we need to either bump the node version or quarantine/skip the astro tests. |
.changeset/strong-ways-destroy.md
Outdated
@@ -0,0 +1,5 @@ | |||
--- | |||
"create-cloudflare": minor |
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.
No longer needs a changeset?
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.
done 👍
Includes a couple of changes to hopefully address flakiness in e2e tests:
next
for now. These test suites will still run but won't block merges.Reviewer is to perform the following, as applicable:
Note for PR author:
We want to celebrate and highlight awesome PR review! If you think this PR received a particularly high-caliber review, please assign it the label
highlight pr review
so future reviewers can take inspiration and learn from it.