Skip to content

fix(no-top-level-browser-globals): false positive for {#if browser} #1252

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

Merged
merged 3 commits into from
Jun 20, 2025

Conversation

ota-meshi
Copy link
Member

fixes #1251

Copy link

changeset-bot bot commented Jun 20, 2025

🦋 Changeset detected

Latest commit: 2c840d8

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

This PR includes changesets to release 1 package
Name Type
eslint-plugin-svelte 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

Copy link
Contributor

github-actions bot commented Jun 20, 2025

Try the Instant Preview in Online Playground

ESLint Online Playground

Install the Instant Preview to Your Local

npm i https://pkg.pr.new/eslint-plugin-svelte@2c840d8

Published Instant Preview Packages:

View Commit

@baseballyama
Copy link
Member

Oh CI is failed.

@ota-meshi ota-meshi force-pushed the fix-top-level-global branch from 00bfef3 to 5ee87e1 Compare June 20, 2025 01:07
@ota-meshi
Copy link
Member Author

Thanks for the quick review! CI passed so I'll merge it.

@ota-meshi ota-meshi merged commit 5db956e into main Jun 20, 2025
17 checks passed
@ota-meshi ota-meshi deleted the fix-top-level-global branch June 20, 2025 01:15
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.

False positive for no-top-level-browser-globals in markup
2 participants