-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
feat(dev): stabilize v2 dev server #6615
Conversation
🦋 Changeset detectedLatest commit: b640a6b The changes in this PR will be included in the next version bump. This PR includes changesets to release 18 packages
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 |
8cf850d
to
6b53573
Compare
v2_dev
🤖 Hello there, We just published version Thanks! |
can confirm, rebuild time goes down from 1.9 to 1.4s。 |
🤖 Hello there, We just published version Thanks! |
I been noticing random app crashes with v0.0.0-nightly-aa896a4-20230619 and They don't seem to happen with I'm not able to reproduce this error consistently enough to nail down its cause, unfortunately. Will report back if anything changes. |
@xHomu can you |
@pcattori This is on an CJS app, so we're using require(BUILD_DR) instead: https://github.com/manawiki/starrail/blob/main/core.server.ts Output from adding console.log(require(BUILD_DIR)): EDITSeems to be working fine? |
@pcattori I'm not sure if it's entirely related, but I noticed another weird bug introduced between Weird .css.map error with chrome devtools on: Seems like it's expecting a sourcemap file for .css as well? This one is reproduceable, you should able build this locally if you still have the .env I sent from before https://github.com/manawiki/starrail/tree/remix_v1.18_pre_bug Above error shows on Windows with devtools on, switch to |
@xHomu if you are still encountering these issues, could you create separate GitHub issues? |
🤖 Hello there, We just published version Thanks! |
See new docs