QwikDev / qwik

Instant-loading web apps, without effort
https://qwik.dev
MIT License
20.87k stars 1.31k forks source link

fix(dev): don't lowercase segment filenames #7003

Closed wmertens closed 4 weeks ago

wmertens commented 1 month ago

This caused lookups of parents to fail due to the case not matching the case on disk.

There's no reason to lowercase the segment names since they are virtual. It also makes the source panel in the devtools nicer to read.

changeset-bot[bot] commented 1 month ago

🦋 Changeset detected

Latest commit: c7aa4e6355632be2d7c7bb9f093efa2385de259c

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

This PR includes changesets to release 4 packages | Name | Type | | --------------------- | ----- | | @builder.io/qwik | Patch | | eslint-plugin-qwik | Patch | | @builder.io/qwik-city | Patch | | create-qwik | 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

pkg-pr-new[bot] commented 1 month ago

Open in Stackblitz

npm i https://pkg.pr.new/@builder.io/qwik@7003
npm i https://pkg.pr.new/@builder.io/qwik-city@7003
npm i https://pkg.pr.new/eslint-plugin-qwik@7003
npm i https://pkg.pr.new/create-qwik@7003

commit: c7aa4e6

github-actions[bot] commented 1 month ago
built with Refined Cloudflare Pages Action

âš¡ Cloudflare Pages Deployment

Name Status Preview Last Commit
qwik-docs ✅ Ready (View Log) Visit Preview c7aa4e6355632be2d7c7bb9f093efa2385de259c