Skip to content

Conversation

@benmccann
Copy link
Member

@benmccann benmccann commented Nov 22, 2022

fixes #7557

no idea why lint is failing on this, but it seems unrelated

@changeset-bot
Copy link

changeset-bot bot commented Nov 22, 2022

🦋 Changeset detected

Latest commit: 94c0188

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

This PR includes changesets to release 1 package
Name Type
@sveltejs/kit 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

@Rich-Harris
Copy link
Member

lint failure is caused by #7771

@Rich-Harris
Copy link
Member

this doesn't seem to have changed anything:

image

@benmccann
Copy link
Member Author

Hmm. This is what I see in Firefox with adapter-static (doing a build and then running the static site with python -m http.server)

Screenshot from 2022-11-22 13-20-20

It resolves the console warnings for me as well

@Rich-Harris
Copy link
Member

Weird, I could have sworn I was looking at the preview deployment. Confirmed fixed with the most recent one

@Rich-Harris Rich-Harris merged commit c09712e into sveltejs:master Nov 23, 2022
@benmccann benmccann deleted the modulePreload-disable branch November 23, 2022 04:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

preloaded with link preload was not used within a few seconds. Warning

2 participants