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

Add latest build table to VMR README #47136

Merged
merged 5 commits into from
Feb 26, 2025
Merged

Add latest build table to VMR README #47136

merged 5 commits into from
Feb 26, 2025

Conversation

ViktorHofer
Copy link
Member

@ViktorHofer ViktorHofer commented Feb 26, 2025

Fixes dotnet/source-build#4717

  • Update the svg to be 40px wider (required as the version is now wider with .NET 10 and the UB version is 2 digits longer)
  • This is the table from sdk main
  • Only include the "main" branch for now
  • Link to the "10.0.1xx-ub" aka.ms links for the VMR
  • I'm sure verticals are missing

Let's iterate on this post-merge (I want to have this ready for today's platform talk)

@Copilot Copilot bot review requested due to automatic review settings February 26, 2025 11:13
@ViktorHofer ViktorHofer requested review from a team as code owners February 26, 2025 11:13
@dotnet-issue-labeler dotnet-issue-labeler bot added Area-Infrastructure untriaged Request triage from a team member labels Feb 26, 2025

Choose a reason for hiding this comment

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

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (1)

src/SourceBuild/content/docs/package-table.md:8

  • The table header only defines two columns, but this row contains four cells. Please update the row to match the intended column format.
| **Windows arm** | **N/A** | **N/A** | **N/A** |
--------------------------------------------------------------------------------------
| Platform | main<br>(10.0.x&nbsp;Runtime) |
| :--------- | :----------: |
| **Windows x64** | [![][win-x64-badge-main]][win-x64-version-main]<br>[Installer][win-x64-installer-main] - [Checksum][win-x64-installer-checksum-main]<br>[zip][win-x64-zip-main] - [Checksum][win-x64-zip-checksum-main] |
Copy link
Member

Choose a reason for hiding this comment

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

Version badges appear truncated -
image

Copy link
Member Author

Choose a reason for hiding this comment

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

This gets fixed in this PR but requires a new official build. See the svg change in this PR.

@ViktorHofer
Copy link
Member Author

ViktorHofer commented Feb 26, 2025

@MichaelSimons & @akoeplinger thanks for the feedback. Note that this is a 1:1 copy from sdk. Would you mind if I react to it in a follow-up PR? I want to show this in today's platform talk, including a shoutout for dogfooding the bits.

I'm also OK with addressing the feedback and then force-merging after the current CI run is green.

@akoeplinger
Copy link
Member

sure, I'm fine with that

ViktorHofer and others added 3 commits February 26, 2025 17:06
Co-authored-by: Alexander Köplinger <alex.koeplinger@outlook.com>
Co-authored-by: Michael Simons <msimons@microsoft.com>
@ViktorHofer ViktorHofer merged commit cb818c4 into main Feb 26, 2025
5 of 39 checks passed
@ViktorHofer ViktorHofer deleted the AddBuildTableToVMR branch February 26, 2025 16:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-Infrastructure untriaged Request triage from a team member
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Include product download links in VMR's readme
3 participants