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

feat: restore release table headings, add platform #357

Merged
merged 1 commit into from
Aug 29, 2023
Merged

Conversation

mistydemeo
Copy link
Contributor

@mistydemeo mistydemeo commented Aug 23, 2023

This brings back column headings that had been removed in 0.1.0, and adds an extra platform column. Right now those are displayed as triples rather than as the human-formatted ones that oranda uses. They're now printed as pretty human-readable names like oranda!

Fixes #356.

@mistydemeo mistydemeo force-pushed the column_headers branch 2 times, most recently from 375d997 to 2690f8f Compare August 24, 2023 23:33
@mistydemeo mistydemeo marked this pull request as draft August 24, 2023 23:34
@mistydemeo
Copy link
Contributor Author

Switching to draft while it's using a prerelease axoproject.

Copy link
Contributor

@Gankra Gankra left a comment

Choose a reason for hiding this comment

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

nice and simple

Comment on lines -1811 to +1819
gh_body.push_str("| | |\n");
gh_body.push_str("|--------|--------|\n");
gh_body.push_str("| File | Platform | Checksum |\n");
gh_body.push_str("|--------|----------|----------|\n");
Copy link
Contributor

Choose a reason for hiding this comment

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

huh interesting, I apparently didn't make this code tolerant of the fact that checksums can be disabled. Whelp, pre-existing issue.

@mistydemeo mistydemeo marked this pull request as ready for review August 28, 2023 22:52
@mistydemeo mistydemeo merged commit c6fb145 into main Aug 29, 2023
@mistydemeo mistydemeo deleted the column_headers branch August 29, 2023 00:00
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.

reintroduce column headers, add human-readable platform column
2 participants