Skip to content

Fix printing Helix workitems count for Android#95061

Merged
akoeplinger merged 1 commit intodotnet:mainfrom
akoeplinger:simplify-helix-workitems
Nov 22, 2023
Merged

Fix printing Helix workitems count for Android#95061
akoeplinger merged 1 commit intodotnet:mainfrom
akoeplinger:simplify-helix-workitems

Conversation

@akoeplinger
Copy link
Copy Markdown
Member

I noticed that we printed "Done building Helix work items. Work item count: 0" in the Send to Helix step for Android jobs. This is because we only looked at Apple-specific XHarnessAppBundleToTest items in sendtohelix-mobile.proj.

Simplified this logic a bit so we can print in sendtohelixhelp.proj instead together with the non-mobile message.

I noticed that we printed "Done building Helix work items. Work item count: 0" in the Send to Helix step for Android jobs.
This is because we only looked at Apple-specific XHarnessAppBundleToTest items in sendtohelix-mobile.proj.

Simplified this logic a bit so we can print in sendtohelixhelp.proj instead together with the non-mobile message.
@ghost
Copy link
Copy Markdown

ghost commented Nov 21, 2023

Tagging subscribers to this area: @directhex
See info in area-owners.md if you want to be subscribed.

Issue Details

I noticed that we printed "Done building Helix work items. Work item count: 0" in the Send to Helix step for Android jobs. This is because we only looked at Apple-specific XHarnessAppBundleToTest items in sendtohelix-mobile.proj.

Simplified this logic a bit so we can print in sendtohelixhelp.proj instead together with the non-mobile message.

Author: akoeplinger
Assignees: -
Labels:

area-Infrastructure-mono

Milestone: -

@ghost ghost assigned akoeplinger Nov 21, 2023
@akoeplinger akoeplinger merged commit e440ebc into dotnet:main Nov 22, 2023
@akoeplinger akoeplinger deleted the simplify-helix-workitems branch November 22, 2023 09:32
@github-actions github-actions Bot locked and limited conversation to collaborators Dec 23, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants