Skip to content

Fix ResX Code Generation #663

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

Merged
merged 1 commit into from
May 7, 2019
Merged

Fix ResX Code Generation #663

merged 1 commit into from
May 7, 2019

Conversation

rladuca
Copy link
Member

@rladuca rladuca commented May 7, 2019

Due to a change in Arcade in GenerateResxSource.targets, we need to set GenerateResxSource to true so that codegen runs on our EmbeddedResources. This is blocking some DARC PRs with the latest Arcade changes.

fixes #662

…et GenerateResxSource to true so that codegen runs on our EmbeddedResources. This is blocking some DARC PRs with the latest Arcade changes.
@rladuca rladuca merged commit a0cf8b8 into master May 7, 2019
Copy link
Member

@vatsan-madhavan vatsan-madhavan left a comment

Choose a reason for hiding this comment

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

Are there situations where we’d want this off by default?

Can we make it so that this is controlled on our end by one and not two knobs ?

@rladuca
Copy link
Member Author

rladuca commented May 7, 2019

@vatsan-madhavan There is only one knob, the GenerateResxSource property. I left GenerateSource set EmbeddedResource since, otherwise, we'd have to wait for DARC flow to get things working. With both in there, we get current builds succeeding + builds with the new Arcade stuff. Once that is pushed everywhere I can remove GenerateSource if we want.

@rladuca rladuca deleted the dev/roladuca/fixresxgen branch June 4, 2019 23:55
@ghost ghost locked as resolved and limited conversation to collaborators Apr 17, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Fix ResX code generation due to changes in Arcade
3 participants