Skip to content

Conversation

@MarijnS95
Copy link
Contributor

Fixes #1838

@MarijnS95
Copy link
Contributor Author

TODO: Need to check how many more things need this.

@mikebattista
Copy link
Collaborator

TODO: Need to check how many more things need this.

Is this PR complete? Have you validated your original test case against your changes?

@MarijnS95
Copy link
Contributor Author

@mikebattista no, unfortunately I have not set up the repro case yet nor validated all the other .Split() calls that seem like they could benefit from the same change but am not sure about. Not to speak of the many remaining untouched .Split() calls. Guess I'm in a bit over my head.

@MarijnS95
Copy link
Contributor Author

In addition, I don't actually remember (if I ever did this at all) how to turn <Project Sdk="Microsoft.Windows.WinmdGenerator/0.59.13-preview"> into something that imports or replaces the "Sdk" with my own (local checkout of this) PR?

@riverar
Copy link
Collaborator

riverar commented Mar 3, 2025

I just hit this again in another project, very annoying. Let's get this merged in and we can address any issues if they come up.

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.

Trim whitespace when parsing <TraverseFiles> contents?

3 participants