-
Notifications
You must be signed in to change notification settings - Fork 6k
Adding ms.update-cycle #46988
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
base: main
Are you sure you want to change the base?
Adding ms.update-cycle #46988
Conversation
#assign-reviewer:dominicallen |
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.
Can we just add it at the pos-for-net
folder level in docfx.json?
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.
@gewarren Definitely, if this is what you want. The script respected the current approach used in the repo, and UpdateFrequency5 was set at the file-level. Do you want to also set UpdateFrequency5 at the pos-for-net folder level and remove it from the files?
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.
Sure, but do we even need to retain that metadata?
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.
Is it currently used in any of your reporting? @dominicallen would you know?
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.
I'm not aware that it's used in any reporting, but might be worth keeping for a short while to make sure. We can follow-up once we've had a successful report run to scrub the old metadata from files
Co-authored-by: Genevieve Warren <[email protected]>
Summary
@dominicallen
Updating freshness metadata as per https://dev.azure.com/msft-skilling/Content/_workitems/edit/434161/?view=edit
For list of update scenarios, see: https://microsoft.sharepoint.com/:x:/t/cfit/ESbbDqmHXspNsU6uJGadREIB9vChqX0wa-HOeqbOb3JXeQ?e=PMIJVi&nav=MTVfezgwRTFDMDQ3LUJBRDctNENDQS1CNEFGLThDNDRCNkFCRTNFQX0
Internal previews
Toggle expand/collapse
Note
This table shows preview links for the 30 files with the most changes. For preview links for other files in this PR, select OpenPublishing.Build Details within checks.