Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added automation for Doc.MS release step #965

Merged
merged 2 commits into from
Sep 11, 2020
Merged

Added automation for Doc.MS release step #965

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
9f0a2b0
Added automation for Doc.MS release step
sima-zhu Sep 4, 2020
97d8eb1
Update the label to auto-merge
sima-zhu Sep 10, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions eng/common/pipelines/templates/steps/docs-metadata-release.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@ parameters:
TargetDocRepoOwner: ''
PRBranchName: 'smoke-test-rdme'
SourceBranchName: 'smoke-test'
PRLabels: 'automation'
Copy link
Member

Choose a reason for hiding this comment

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

@scbedd should we go ahead and also start applying the 'auto-merge' label?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sync with @scbedd , the DocMs PR does not have any auto-merge label created yet. Will have auto-merge as a placeholder for now, but it might not function like it should be. Meanwhile, I will have a simple test to verify the script did not fail if the label does not exist.

Copy link
Member

Choose a reason for hiding this comment

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

After chatting with @scbedd I think we want to just replace 'automation' with 'auto-merge'.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, forgot to update. Thanks!

ArtifactName: ''
Language: ''
DocRepoDestinationPath: '' #usually docs-ref-services/
Expand Down Expand Up @@ -81,6 +82,7 @@ steps:
ScriptDirectory: ${{ parameters.WorkingDirectory }}/${{ parameters.ScriptDirectory }}
GHReviewersVariable: ${{ parameters.GHReviewersVariable }}
GHTeamReviewersVariable: ${{ parameters.GHTeamReviewersVariable }}
PRLabels: ${{ parameters.PRLabels }}

- ${{if ne( parameters['OnboardingBranch'], '')}}:
- pwsh: |
Expand Down