Skip to content

Comments

Refactor os, arch, and libc information into a shared Platform type#15027

Merged
zanieb merged 7 commits intomainfrom
zb/platform-sorting
Aug 13, 2025
Merged

Refactor os, arch, and libc information into a shared Platform type#15027
zanieb merged 7 commits intomainfrom
zb/platform-sorting

Conversation

@zanieb
Copy link
Member

@zanieb zanieb commented Aug 2, 2025

Addresses this outstanding item from a previous review #13724 (comment)

I'm interested in this now for consolidating some logic in #12731

@zanieb zanieb temporarily deployed to uv-test-registries August 2, 2025 12:40 — with GitHub Actions Inactive
@zanieb zanieb force-pushed the zb/platform-sorting branch from 03aff1f to 4781989 Compare August 2, 2025 12:53
@zanieb zanieb temporarily deployed to uv-test-registries August 2, 2025 12:55 — with GitHub Actions Inactive
@zanieb zanieb force-pushed the zb/platform-sorting branch from 4781989 to 2d617a8 Compare August 2, 2025 13:23
@zanieb zanieb temporarily deployed to uv-test-registries August 2, 2025 13:26 — with GitHub Actions Inactive
@zanieb zanieb force-pushed the zb/platform-sorting branch from 2d617a8 to f6f04df Compare August 5, 2025 12:20
@zanieb zanieb temporarily deployed to uv-test-registries August 5, 2025 12:23 — with GitHub Actions Inactive
@zanieb zanieb temporarily deployed to uv-test-registries August 5, 2025 12:29 — with GitHub Actions Inactive
@zanieb zanieb force-pushed the zb/platform-sorting branch from 2741e1c to d93f546 Compare August 5, 2025 12:37
@zanieb zanieb temporarily deployed to uv-test-registries August 5, 2025 12:44 — with GitHub Actions Inactive
@zanieb zanieb force-pushed the zb/platform-sorting branch from d93f546 to 1dc64af Compare August 5, 2025 18:19
@zanieb zanieb temporarily deployed to uv-test-registries August 5, 2025 18:21 — with GitHub Actions Inactive
@zanieb zanieb force-pushed the zb/platform-sorting branch from 1dc64af to e88b548 Compare August 5, 2025 20:02
@zanieb zanieb temporarily deployed to uv-test-registries August 5, 2025 20:13 — with GitHub Actions Inactive
@zanieb zanieb force-pushed the zb/platform-sorting branch from 0c4febc to 08f7c81 Compare August 5, 2025 20:25
@zanieb zanieb temporarily deployed to uv-test-registries August 5, 2025 20:26 — with GitHub Actions Inactive
@zanieb zanieb marked this pull request as ready for review August 5, 2025 21:29
@zanieb zanieb marked this pull request as draft August 5, 2025 21:33
@zanieb zanieb marked this pull request as ready for review August 7, 2025 20:14
@zanieb zanieb temporarily deployed to uv-test-registries August 7, 2025 20:36 — with GitHub Actions Inactive
@zanieb zanieb temporarily deployed to uv-test-publish August 7, 2025 20:36 — with GitHub Actions Inactive
@zanieb zanieb temporarily deployed to uv-test-registries August 8, 2025 15:11 — with GitHub Actions Inactive
@zanieb zanieb temporarily deployed to uv-test-publish August 8, 2025 15:11 — with GitHub Actions Inactive
@zanieb zanieb force-pushed the zb/platform-sorting branch from 0e07899 to a203c1f Compare August 8, 2025 15:33
@zanieb zanieb temporarily deployed to uv-test-registries August 8, 2025 15:35 — with GitHub Actions Inactive
@zanieb zanieb temporarily deployed to uv-test-registries August 12, 2025 21:57 — with GitHub Actions Inactive
@zanieb zanieb temporarily deployed to uv-test-publish August 12, 2025 21:57 — with GitHub Actions Inactive
@zanieb zanieb force-pushed the zb/platform-sorting branch from c9329db to 0b21a9c Compare August 12, 2025 22:25
@zanieb zanieb added the internal A refactor or improvement that is not user-facing label Aug 12, 2025
@zanieb zanieb temporarily deployed to uv-test-registries August 12, 2025 22:27 — with GitHub Actions Inactive
@zanieb zanieb merged commit 78c8c71 into main Aug 13, 2025
95 checks passed
@zanieb zanieb deleted the zb/platform-sorting branch August 13, 2025 14:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal A refactor or improvement that is not user-facing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant