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

Update build.yml to test including process dumps #201

Closed
wants to merge 11 commits into from

Conversation

michael-hawker
Copy link
Member

@michael-hawker michael-hawker commented Jul 31, 2024

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.

@michael-hawker michael-hawker changed the base branch from wasdk/1.6 to main July 31, 2024 21:20
@michael-hawker michael-hawker force-pushed the dev/test-process-dumps branch from 67fc76c to ff1ae97 Compare July 31, 2024 21:21
@Arlodotexe Arlodotexe changed the base branch from main to align-heads-with-multitarget July 31, 2024 21:25
@Arlodotexe Arlodotexe changed the base branch from align-heads-with-multitarget to wasdk/1.6 July 31, 2024 21:26
@Arlodotexe
Copy link
Member

Oh, I hadn't noticed you manually changed the base branch. I'll revert my change.

@Arlodotexe Arlodotexe changed the base branch from wasdk/1.6 to main July 31, 2024 21:27
@michael-hawker
Copy link
Member Author

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.

@michael-hawker
Copy link
Member Author

Hitting microsoft/slngen#451 when turning diagnostics on, so will turn slngen flag carrying that over.

@Arlodotexe
Copy link
Member

Migrating to CommunityToolkit/Windows#493

@Arlodotexe Arlodotexe closed this Sep 6, 2024
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.

2 participants