Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow using ThisAssembly.Resources and ThisAssembly.Strings SxS #225

Merged
merged 1 commit into from
Apr 8, 2023
Merged

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.

1 participant