Skip to content

Conversation

InkedCat
Copy link

This PR adds support for Windows environments.

It:

  • Forces \n as the end-of-line character
  • Normalizes paths whenever possible to stay cross-platform
  • Enables Windows runners in NodeCI

closes #1335

Copy link

changeset-bot bot commented Sep 11, 2025

⚠️ No Changeset found

Latest commit: 4463dde

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copy link
Contributor

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@4463dde

Published Instant Preview Packages:

View Commit

Copy link
Member

@baseballyama baseballyama left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you so much! Can you add changeset like this?
(I tried to push directly to your branch, but I didn’t have permission to do so.)

fix: properly support Windows in `no-unused-props` rule
fix: properly support Windows in `valid-style-parse` rule
fix: properly support Windows in `nno-unnecessary-condition` rule

@InkedCat
Copy link
Author

InkedCat commented Sep 13, 2025

Thank you so much! Can you add changeset like this? (I tried to push directly to your branch, but I didn’t have permission to do so.)

fix: properly support Windows in `no-unused-props` rule
fix: properly support Windows in `valid-style-parse` rule
fix: properly support Windows in `nno-unnecessary-condition` rule

Hello ! Hope you have a good day.
I've tried to run pnpm update before adding the changesets, but it seems there are still some bugs related to Windows style paths in the package tools scripts. I'll need to change the writeAndFormat function signature to accept URL's, a URL will handle the conversion for every platform correctly.

Should I push the changesets directly without updating or make the tools cross platform ?

@baseballyama
Copy link
Member

Should I push the changesets directly without updating or make the tools cross platform ?

I’d be very happy if you could take care of it, but if you don’t have the time, I’d appreciate it if you could at least change the PR settings so that I can push.

image

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.

svelte/no-unused-props misinterprets props as internal properties
2 participants