Skip to content

Remove AAD credentials from Service-Level-Readme-Automation.ps1 invocation#32316

Merged
danieljurek merged 1 commit intomainfrom
djurek/codeowners-automation-cleanup
Oct 3, 2023
Merged

Remove AAD credentials from Service-Level-Readme-Automation.ps1 invocation#32316
danieljurek merged 1 commit intomainfrom
djurek/codeowners-automation-cleanup

Conversation

@danieljurek
Copy link
Member

No description provided.

@danieljurek danieljurek enabled auto-merge (squash) October 2, 2023 22:16
@JimSuplizio JimSuplizio self-requested a review October 3, 2023 15:29
@danieljurek danieljurek merged commit 2f70b51 into main Oct 3, 2023
@danieljurek danieljurek deleted the djurek/codeowners-automation-cleanup branch October 3, 2023 15:29
inputs:
pwsh: true
filePath: eng/common/scripts/Service-Level-Readme-Automation.ps1
arguments: >-
Copy link
Member

Choose a reason for hiding this comment

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

@danieljurek lets also be sure to remove these variables from the docindex pipelines.

Copy link
Member Author

Choose a reason for hiding this comment

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

Copy link
Member

@weshaggard weshaggard Oct 3, 2023

Choose a reason for hiding this comment

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

Copy link
Member Author

Choose a reason for hiding this comment

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

Actually done. I removed the secrets from the variable group as other secrets in there are still needed.

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.

3 participants