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

Fix test result publishing #1244

Merged
merged 3 commits into from
Apr 25, 2024
Merged

Fix test result publishing #1244

merged 3 commits into from
Apr 25, 2024

Conversation

lbussell
Copy link
Contributor

@lbussell lbussell commented Apr 17, 2024

See #1246

  • Testing this fix in PR validation since it affects public builds
  • Rearrange docker copy and test results check commands

Copy link

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.

@lbussell
Copy link
Contributor Author

/azp run

Copy link

Azure Pipelines failed to run 1 pipeline(s).

@lbussell
Copy link
Contributor Author

The correct solution to the issue of having no test results to publish is #1250.

@lbussell lbussell marked this pull request as ready for review April 25, 2024 15:40
@lbussell lbussell requested a review from a team as a code owner April 25, 2024 15:40
@lbussell
Copy link
Contributor Author

This change should go in as-is and #1250 should be addressed separately so that test results are available again as soon as possible.

@lbussell lbussell enabled auto-merge (squash) April 25, 2024 16:29
@lbussell lbussell merged commit 8fa03e2 into dotnet:main Apr 25, 2024
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants