Skip to content

Always use posix path separators#20

Merged
Bessonov merged 1 commit intoBessonov:masterfrom
xpdota:fix-windows-path
Mar 1, 2026
Merged

Always use posix path separators#20
Bessonov merged 1 commit intoBessonov:masterfrom
xpdota:fix-windows-path

Conversation

@xpdota
Copy link
Copy Markdown
Contributor

@xpdota xpdota commented Feb 27, 2026

Fixes #19

Forces POSIX paths on every platform.

Ran tests on windows and WSL.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes Issue #19, where on Windows the tool was generating TypeScript project references with backslash (\\) path separators instead of forward slash (/) POSIX separators, causing spurious "missing" and "obsolete" reference diffs. The fix targets the one location in src/index.ts where relative paths are computed and stored as TS project references.

Changes:

  • Converts the computed relative path to POSIX format before storing it as a TypeScript project reference, to ensure forward slashes are used on all platforms.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/index.ts Outdated
@Bessonov Bessonov assigned xpdota and unassigned Bessonov Mar 1, 2026
@xpdota xpdota force-pushed the fix-windows-path branch from d4b4fd3 to 8c5add9 Compare March 1, 2026 16:12
@Bessonov Bessonov merged commit 725c256 into Bessonov:master Mar 1, 2026
@Bessonov
Copy link
Copy Markdown
Owner

Bessonov commented Mar 1, 2026

Thanks for your contributions! I still need to make a few more changes before the release, like updating dependencies. I think it'll take a few days.

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.

On Windows, set-project-references tries to replace / with \\

3 participants