Skip to content

Conversation

@XiaofeiCao
Copy link

@XiaofeiCao XiaofeiCao force-pushed the latest_release_date branch from 3aee9ce to 62deadc Compare May 12, 2025 07:47
@XiaofeiCao XiaofeiCao marked this pull request as ready for review May 12, 2025 08:33
Copilot AI review requested due to automatic review settings May 12, 2025 08:33
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds support for the new LatestPreviewDate field to the -packages.csv files to address issue #8819.

  • Updated eng/scripts/Update-Release-Versions.ps1 to assign LatestPreviewDate using the first preview version, if available.
  • Enhanced eng/scripts/PackageList-Helpers.ps1 to include the LatestPreviewDate property when creating and cloning package objects.
  • Modified eng/README.md and various CSV files (ios, cpp, c, android) to document and include the LatestPreviewDate column.

Reviewed Changes

Copilot reviewed 13 out of 13 changed files in this pull request and generated 1 comment.

File Description
eng/scripts/Update-Release-Versions.ps1 Added logic to compute and assign LatestPreviewDate from preview versions.
eng/scripts/PackageList-Helpers.ps1 Updated package creation and cloning functions to support LatestPreviewDate.
eng/README.md Updated documentation to include details on the LatestPreviewDate field.
_data/releases/latest/*.csv Modified CSV files to add the LatestPreviewDate column in the release data.

- `LatestGADate` - This field is used to identify the date of when the latest GA package released.
- `FirstGADate` - This field is used to identify the date of when a new package shipped its first stable release.
- `FirstPreviewDate` - This field is used to identify the date of when a new package shipped its first preview release.
- `LatestPreviewDate` - This field is used to identify the date of when the latest preview package released.
Copy link
Member

Choose a reason for hiding this comment

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

Why is the LatestPreviewDate interesting? They are supposed to be temporary until the next GA so we shouldn't care about this.

@weshaggard
Copy link
Member

@XiaofeiCao closing this PR as I would prefer we not go this route. Talk with me further if you there is more data you think make this work needed.

@weshaggard weshaggard closed this May 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[EngSys] Add LatestPreviewDate to -packages.csv file

3 participants