Skip to content

fix: Translate Windows backslash path separator to forward slash for …#1377

Open
igarashitm wants to merge 1 commit into
KaotoIO:mainfrom
igarashitm:3212-win-include
Open

fix: Translate Windows backslash path separator to forward slash for …#1377
igarashitm wants to merge 1 commit into
KaotoIO:mainfrom
igarashitm:3212-win-include

Conversation

@igarashitm
Copy link
Copy Markdown
Member

@igarashitm igarashitm commented May 15, 2026

…both metadata file and the resource file

Fixes: KaotoIO/kaoto#3212

Now file selector shows candidates with forward slash as a path separator

Screenshot 2026-05-15 144705

xs:include resolution succeeds on Windows
Screenshot 2026-05-15 144911

Screenshot 2026-05-15 150151

Summary by CodeRabbit

Bug Fixes

  • Fixed file path normalization to ensure consistent behavior across Windows and Unix-based systems for metadata parsing and compatibility with previously saved projects.

Review Change Stack

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 15, 2026

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 87660b89-84d8-4d04-b466-636f15458311

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • ✅ Review completed - (🔄 Check again to review again)
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@sonarqubecloud
Copy link
Copy Markdown

@igarashitm igarashitm marked this pull request as ready for review May 15, 2026 19:25
@igarashitm igarashitm requested a review from djelinek May 15, 2026 19:25
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.

DataMapper: xs:include file path resolution fails on Windows

1 participant