Skip to content

Conversation

@kzu
Copy link
Member

@kzu kzu commented Apr 8, 2023

This fixes an issue where the additional file metadata (as persisted by the generated editorconfig for that) is duplicated and overriden between these two packages since both use EmbeddedResource item to drive the source generator.

Switch to per-generator item metadata and avoid using AdditionalFiles item metadata, which has this problem.

@kzu kzu added the bug Something isn't working label Apr 8, 2023
@kzu kzu changed the title Allow using ThisAssembly.Resources and ThisAssembly.Stirngs SxS Allow using ThisAssembly.Resources and ThisAssembly.Strings SxS Apr 8, 2023
@kzu kzu force-pushed the dev/SxS branch 2 times, most recently from 6cefd19 to 9e281f2 Compare April 8, 2023 21:54
This fixes an issue where the additional file metadata (as persisted by the generated editorconfig for that) is duplicated and overriden between these two packages since both use EmbeddedResource item to drive the source generator.

Switch to per-generator item metadata and avoid using AdditionalFiles item metadata, which has this problem.
@kzu kzu enabled auto-merge (rebase) April 8, 2023 21:54
@kzu kzu merged commit da44b55 into main Apr 8, 2023
@kzu kzu deleted the dev/SxS branch April 8, 2023 22:02
@devlooped devlooped locked and limited conversation to collaborators Sep 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants