-
Notifications
You must be signed in to change notification settings - Fork 11
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
Update build.yml to test including process dumps #201
Conversation
…to head, removed redundant RuntimeIdentifiers definition on libraries.
Needed to help diagnose issue CommunityToolkit/Windows#347
67fc76c
to
ff1ae97
Compare
Oh, I hadn't noticed you manually changed the base branch. I'll revert my change. |
Yeah, I don't think it wouldn't run the CI if I was just targeting your branch, I just wanted to test against the latest WASDK, so grabbed your branch as a starting point. 😊 Don't intend to merge this PR, just use it to collect dumps, and if that works can clean-it up later, but at least want to get dumps to provide platform team to see if they can assist us in understanding what's happening, if we can't. |
Hitting microsoft/slngen#451 when turning diagnostics on, so will turn slngen flag carrying that over. |
Migrating to CommunityToolkit/Windows#493 |
Needed to help diagnose issue CommunityToolkit/Windows#347
built-off AOT
wasdk/1.6
branch, as suspect we've seen an uptick with this issue with 1.6 over 1.5, but it's also more finicky in PR build over integration builds, which is also weird.