-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Conversation
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.
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** |
b4151ae
to
2e0ec35
Compare
6d9c601
to
b2b79d4
Compare
-------------------------------------------------------------------------------------- | ||
| Platform | main<br>(10.0.x 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] | |
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.
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.
This gets fixed in this PR but requires a new official build. See the svg change in this PR.
@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. |
sure, I'm fine with that |
Co-authored-by: Alexander Köplinger <alex.koeplinger@outlook.com> Co-authored-by: Michael Simons <msimons@microsoft.com>
Fixes dotnet/source-build#4717
Let's iterate on this post-merge (I want to have this ready for today's platform talk)