Skip to content

Only check the touched markdown files in PR for the Verify link step #15929

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

Merged
sima-zhu merged 9 commits into from
Oct 16, 2020
Merged

Only check the touched markdown files in PR for the Verify link step #15929

Changes from all commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
ea88d4f
Only check the touched markdown files in PR for the Verify link step
sima-zhu Oct 13, 2020
421a363
Change to base branch insteasd of master
sima-zhu Oct 14, 2020
5d8ba92
add conditions
sima-zhu Oct 15, 2020
f9681e7
check on steps
sima-zhu Oct 15, 2020
1792c26
Update archetype-sdk-client.yml
sima-zhu Oct 15, 2020
6981982
Fixed and test
sima-zhu Oct 16, 2020
cc6b6cb
Merge branch 'checkOnDiff' of https://github.com/sima-zhu/azure-sdk-f…
sima-zhu Oct 16, 2020
d4f2269
Update README.md
sima-zhu Oct 16, 2020
0d654fe
Update README.md
sima-zhu Oct 16, 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
10 changes: 6 additions & 4 deletions eng/pipelines/templates/jobs/archetype-sdk-client.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -66,10 +66,12 @@ jobs:
displayName: "Use Python 3.6"
inputs:
versionSpec: "3.6"
- template: /eng/common/pipelines/templates/steps/verify-links.yml
parameters:
Directory: sdk/${{ parameters.ServiceDirectory }}
CheckLinkGuidance: $true
- ${{ if eq(variables['Build.Reason'], 'PullRequest') }}:
- template: /eng/common/pipelines/templates/steps/verify-links.yml
parameters:
Directory: ''
Urls: (git diff origin/${env:SYSTEM_PULLREQUEST_TARGETBRANCH} HEAD --name-only -- '*.md')
Copy link
Contributor

@pakrym pakrym Oct 16, 2020

Choose a reason for hiding this comment

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

Can't we encapsulate this logic inside /eng/common/pipelines/templates/steps/verify-links.yml ?

Copy link
Contributor Author

@sima-zhu sima-zhu Oct 16, 2020

Choose a reason for hiding this comment

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

We currently don't want to update all lang repo. Some languages repos do not have aggregate-report pipeline setup. So it is less the ability of scanning the entire repo. We only update 4 major langs for now. Once we want to update to all repo, we will move the logic inside of verify-links.yml

CheckLinkGuidance: $true
- script: |
pip install setuptools wheel
pip install doc-warden==$(DocWardenVersion)
Expand Down