-
Notifications
You must be signed in to change notification settings - Fork 187
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.
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
Remove get-codeowners.ps1 from docs automation #7040
Conversation
The following pipelines have been queued for testing: |
The following pipelines have been queued for testing: |
The following pipelines have been queued for testing: |
The following pipelines have been queued for testing: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Daniel,
As per our discussion over teams. I'm good with these changes. For anyone else, there was one requested change which was done with the 5th commit that fixed up the notification-configuration solution which, for some reason, opted to pull in the entire CodeownersParser solution instead of just the library's project like the other consumers do.
Sync eng/common directory with azure-sdk-tools for PR Azure/azure-sdk-tools#7040 See [eng/common workflow](https://github.com/Azure/azure-sdk-tools/blob/main/eng/common/README.md#workflow) --------- Co-authored-by: Daniel Jurek <[email protected]>
Fixes #6461
This also requires changes to docs repos which remove the
author
andms.author
values written into individual markdown files and configuresdocfx.json
to set defaultauthor
andms.author
values for those files.Required changes after this merges:
docindex.yml
for each languagedocfx.json
and removal of author metadata for each docs repoExamples:
ms.date
instead ofms.data
and no author metadata) -- Azure/azure-docs-sdk-java@fef7e42"author"
and"ms.author"
to see values specified indocfx.json
)