Skip to content

Conversation

@lhecker
Copy link
Member

@lhecker lhecker commented Jul 18, 2022

#13368 (bd403dc) failed to increment the VS version
in our compliance pipeline which broke it.

@DHowett
Copy link
Member

DHowett commented Jul 18, 2022

Does it work with this change?

@lhecker lhecker force-pushed the dev/lhecker/fix-compliance branch 4 times, most recently from 9bb6a58 to bd68089 Compare July 18, 2022 21:54
@lhecker lhecker force-pushed the dev/lhecker/fix-compliance branch from bd68089 to 84cd86a Compare July 18, 2022 22:23
@lhecker
Copy link
Member Author

lhecker commented Jul 18, 2022

5th time's the charm. The compliance job now completes successfully.


# Set XES_SERIALPOSTBUILDREADY to run Security and Compliance task once per build
- powershell: Write-Host ##vso[task.setvariable variable=XES_SERIALPOSTBUILDREADY;]true
- powershell: Write-Host "##vso[task.setvariable variable=XES_SERIALPOSTBUILDREADY;]true"
Copy link
Member Author

@lhecker lhecker Jul 18, 2022

Choose a reason for hiding this comment

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

Those quotes were a bit surprising. I don't think they were intentional were they? I took the liberty to fix them...

@DHowett DHowett added the AutoMerge Marked for automatic merge by the bot when requirements are met label Jul 19, 2022
@ghost
Copy link

ghost commented Jul 19, 2022

Hello @DHowett!

Because this pull request has the AutoMerge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

@ghost ghost merged commit d2b7750 into main Jul 19, 2022
@ghost ghost deleted the dev/lhecker/fix-compliance branch July 19, 2022 17:00
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AutoMerge Marked for automatic merge by the bot when requirements are met

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants