Skip to content

FEAT: re-export cibuildwheel's markdown summary - #322

Closed
neutrinoceros wants to merge 1 commit into
OpenAstronomy:mainfrom
neutrinoceros:feat/output-cibw-summary
Closed

neutrinoceros wants to merge 1 commit into
OpenAstronomy:mainfrom
neutrinoceros:feat/output-cibw-summary

Conversation

@neutrinoceros

Copy link
Copy Markdown
Contributor

cibuildwheel 3.x has a really nice feature to summaries its output in markdown.
This re-exposes the summary so it's reusable downstream.
It's a bit non-ideal that users need to do extra work to benefit from it, I'd prefer if it was done automatically but I haven't found a way to do that. For now, opening as a draft, as it's missing extra documentation, but I want to make sure I have the best solution before I write it down.

@neutrinoceros

neutrinoceros commented Nov 3, 2025 •

Copy link
Copy Markdown
Contributor Author

this is working as I expect. See https://github.com/OpenAstronomy/github-actions-workflows/actions/runs/19045799811?pr=322
(scroll past the action graph)

@neutrinoceros
neutrinoceros marked this pull request as ready for review November 3, 2025 18:56
@Cadair

Cadair commented Nov 3, 2025

Copy link
Copy Markdown
Member

This already seems to work for sunpy? https://github.com/sunpy/sunpy/actions/runs/18952164984 What am I missing 😄

@neutrinoceros

Copy link
Copy Markdown
Contributor Author

Wait. Did I hallucinate that it didn't already work ?? I must have been misremembering how it behaved, or Github changed something that made this patch completely unneeded. Either way, I get what I wanted, and for cheaper than I thought, so I see this as an absolute win.

@neutrinoceros
neutrinoceros deleted the feat/output-cibw-summary branch November 3, 2025 20:20
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.

2 participants