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.

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

Review .updated.json usage #7541

Open
ben-polinsky opened this issue Jan 13, 2025 · 0 comments
Open

Review .updated.json usage #7541

ben-polinsky opened this issue Jan 13, 2025 · 0 comments

Comments

@ben-polinsky
Copy link
Contributor

ben-polinsky commented Jan 13, 2025

When building and publishing our documentation, we keep a running map of when pages were last updated. To do this, we keep a map of pages->time last updated, publishing it as a build artifact for consumption by the next documentation CI build.

This has mostly worked, but a couple issues have surfaced:

  1. The last updated times do not always match the file's commit history. For instance, see this page: https://www.itwinjs.org/learning/geolocation/ - the last updated time is 15 May, 2024 yet commit history shows the file hasn't been touched in years

  2. When creating new release branches, we must remember to update the download task branch after an artifact is published from that build. For example, in creating a new branch release/10.x we initially point the branch to release/9.x as that's the most recently published branch. Then, after one run of release/10.x which publishes an updated artifact, we can change the branch to release/10.x. Remembering to do this is error-prone.

@DanRod1999

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant