Skip to content

[vcpkg ci] Update formatting CI#12314

Merged
strega-nil merged 7 commits intomicrosoft:masterfrom
strega-nil:stage-formatting
Jul 7, 2020
Merged

[vcpkg ci] Update formatting CI#12314
strega-nil merged 7 commits intomicrosoft:masterfrom
strega-nil:stage-formatting

Conversation

@strega-nil
Copy link
Copy Markdown
Contributor

@strega-nil strega-nil commented Jul 7, 2020

First, stage the formatting, so that we don't run CI if the formatting is wonky; second, format port manifests in CI with x-format-manifest.

this means that the port CI won't run if the formatting failed
Comment thread scripts/azure-pipelines/azure-pipelines.yml
Comment thread scripts/azure-pipelines/azure-pipelines.yml Outdated
@strega-nil strega-nil changed the title [vcpkg ci] Formatting should be a stage [vcpkg ci] Update formatting CI Jul 7, 2020
Comment thread scripts/azure-pipelines/windows/Check-CxxFormatting.ps1 Outdated
@strega-nil strega-nil requested a review from BillyONeal July 7, 2020 18:36
Comment thread scripts/azure-pipelines/windows/Check-ManifestFormatting.ps1 Outdated
Comment thread scripts/azure-pipelines/windows/Check-CxxFormatting.ps1 Outdated
Comment thread scripts/azure-pipelines/windows/Check-ManifestFormatting.ps1 Outdated
@strega-nil strega-nil merged commit 9606917 into microsoft:master Jul 7, 2020
@strega-nil strega-nil deleted the stage-formatting branch July 16, 2020 17:45
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