Conversation
chidozieononiwu
commented
Sep 22, 2021
chidozieononiwu
commented
- Adding common policheck steps to enable policheck run on PR validations.
|
The following pipelines have been queued for testing: |
| ExclusionDataBaseFileName: '' | ||
| TargetDirectory: '' | ||
| PublishAnalysisLogs: false | ||
| PoliCheckBlobSAS: '$(azuresdk-policheck-blob-SAS)' |
There was a problem hiding this comment.
Do we need to add variable group 95 to the pipeline generators for this?
There was a problem hiding this comment.
How did you determine the number 95?
There was a problem hiding this comment.
I was wondering that as well :) I assume he looked up the variable group that defines this.
There was a problem hiding this comment.
I tried that, I still didn't see any number anywhere.
There was a problem hiding this comment.
The only way I have found to find the variable group id is to look at the URL when you are viewing a variable group.
There was a problem hiding this comment.
Thanks @weshaggard . You can also find it using azure cli az pipelines variable-group list --group-name "azuresdkartifacts SAS tokens". But Url is definitly easier.
chidozieononiwu
force-pushed
the
AddCommonPoliCheckSteps
branch
from
caca134 to
07458df
Compare
|
The following pipelines have been queued for testing: |
| TargetDirectory: '' | ||
| PublishAnalysisLogs: false | ||
| PoliCheckBlobSAS: "$(azuresdk-policheck-blob-SAS)" | ||
| UePath: "$(Build.SourcesDirectory)/eng/guardian-tools/policheck/PolicheckExclusions.xml" |
There was a problem hiding this comment.
What does Ue mean? Should we just call this ExclusionFilePath
chidozieononiwu
force-pushed
the
AddCommonPoliCheckSteps
branch
from
07458df to
1956e35
Compare
|
The following pipelines have been queued for testing: |
|
Hello @azure-sdk! Because this pull request has the 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 (
|
ghost
merged commit 4 participants
