Skip to content

Conversation

@ntoll
Copy link
Member

@ntoll ntoll commented Nov 26, 2025

Description

This addresses #2346 discussions about including PyScript version information. To this end the offline.zip file is renamed to offline_<CALVER>.zip (e.g. offline_2025.11.3.zip) so it's easy to identify which version of PyScript is contained within the archive.

Changes

  • Updated the publish-release GitHub action to rename the file before it's attached / pushed to the release.
  • Updated the HTML template used to generate the release page so it's pointing at the right filename.
  • Updated the sed command so replacement is global within a line (the g flag).
  • Tidied up aspects of the template HTML.

Checklist

  • I have checked make build works locally.
  • I have created / updated documentation for this change (if applicable).

@ntoll ntoll requested a review from WebReflection November 26, 2025 12:36
Copy link
Contributor

@WebReflection WebReflection left a comment

Choose a reason for hiding this comment

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

looks good to me

&lt;/body&gt;
&lt;/html&gt;</pre>
&lt;/html&gt;</pre
>
Copy link
Contributor

Choose a reason for hiding this comment

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

what is this? ... I'd add a \n before the ending </pre>

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 is pre-commit hook messing with HTML. :-/ Will look into this.

<a href="offline__VERSION_.zip">offline__VERSION_.zip</a> (PyScript _VERSION_
zipped up for offline use)
<a href="offline__VERSION_.zip">offline__VERSION_.zip</a>
(PyScript _VERSION_ zipped up for offline use)
Copy link
Contributor

Choose a reason for hiding this comment

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

this looks fine, it will result into just a space once visualized

Copy link
Member Author

Choose a reason for hiding this comment

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

Yeah, again pre-commit hook mangled this. 🤦

@ntoll ntoll merged commit 90ae3ce into main Nov 26, 2025
1 check passed
@ntoll ntoll deleted the rename-offline-zip branch November 26, 2025 15:02
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.

3 participants