Skip to content

Diff TestGroupName for project templates tests#42214

Merged
DamianEdwards merged 3 commits into
mainfrom
damianedwards/project-template-tests-group-name-fix
Jun 16, 2022
Merged

Diff TestGroupName for project templates tests#42214
DamianEdwards merged 3 commits into
mainfrom
damianedwards/project-template-tests-group-name-fix

Conversation

@DamianEdwards
Copy link
Copy Markdown
Member

Attempt to fix project templates tests not showing up in CI build results.

@DamianEdwards DamianEdwards requested a review from Pilchie as a code owner June 15, 2022 18:57
@ghost ghost added the area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates label Jun 15, 2022
@DamianEdwards DamianEdwards requested a review from dougbu June 15, 2022 18:58
@TanayParikh TanayParikh requested review from a team and BrennanConroy June 15, 2022 19:58
Copy link
Copy Markdown
Member

@BrennanConroy BrennanConroy left a comment

Choose a reason for hiding this comment

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

@dougbu
Copy link
Copy Markdown
Contributor

dougbu commented Jun 15, 2022

Correct @BrennanConroy. Sorry @DamianEdwards that this wasn't the right fix. This is helpful but changing that one line might be enough.

(https://dev.azure.com/dnceng/public/_build/results?buildId=1827055&view=logs&j=3f6d4e0f-1b71-56b5-361e-d95b6e6da15a&t=e4ee6667-9b0a-5a6e-4e24-bcba939f2af2 doesn't mention the word "templates" at all. Confirmed same checking a couple of the Helix work item lists.)

@DamianEdwards DamianEdwards merged commit 8897c29 into main Jun 16, 2022
@DamianEdwards DamianEdwards deleted the damianedwards/project-template-tests-group-name-fix branch June 16, 2022 03:37
@ghost ghost added this to the 7.0-preview6 milestone Jun 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants