Skip to content

Auto 'pretranspile' package on release creation#47

Merged
confused-Techie merged 7 commits intomasterfrom
auto-transpile
Mar 31, 2026
Merged

Auto 'pretranspile' package on release creation#47
confused-Techie merged 7 commits intomasterfrom
auto-transpile

Conversation

@confused-Techie
Copy link
Copy Markdown
Member

@confused-Techie confused-Techie commented Mar 30, 2026

Previously, to pretranspile a package it was a combination of dark arts and arcane magic thought up by the deranged, that was decoded by @mauricioszabo and later transcribed by @DeeDeeG.

Reference:

This PR aims to codify and automate it.

This PR adds a GHA workflow that on a release creation will automatically create a tag of the same version appended with -pretranspile, to match our current conventions.

Keeping with best practices of using our org defined NodeJS version that'll match Pulsar's current version, and unfortunately a hardcoded Electron version that we will have to keep updated with Pulsar, as well as lots of inspiration taken from our pulsar-release-workflow, pulsar-chocolatey, and action-pulsar-dependency, we should have something that "Just Works ™" here.

Only real way to test it is to run it, so with the upcoming PR from @DeeDeeG to bump some dependencies to resolve issues I'm facing in pulsar-edit/pulsar#1508, it'll be the perfect time to test.

Copy link
Copy Markdown
Member

@DeeDeeG DeeDeeG left a comment

Choose a reason for hiding this comment

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

Thank you for being so thoughtful / proactive as to make this into an automation!

My thoughts on looking this over, but thanks again for the work!

@DeeDeeG
Copy link
Copy Markdown
Member

DeeDeeG commented Mar 30, 2026

Previously, to pretranspile a package it was a combination of dark arts and arcane magic thought up by the deranged, that was decoded by @DeeDeeG.

Credit goes to @mauricioszabo, figuring out how to do this was all him. I merely asked him how he did it and took notes, and the wrote it out in those commit messages as docs. I should have mentioned him as the source of info, but I am quick to remind folks and tell that part of the story if anyone implies I came up with it. (Which in hindsight it looks as if something I invented if you don't happen to know it was actually @mauricioszabo, so of course people think that! So I correct the record when it comes up.)

Co-authored-by: DeeDeeG <DeeDeeG@users.noreply.github.com>
@confused-Techie
Copy link
Copy Markdown
Member Author

@DeeDeeG I was not aware that mauricio was the one to originally discover this! Good fun fact and shoutout

Copy link
Copy Markdown
Member

@DeeDeeG DeeDeeG left a comment

Choose a reason for hiding this comment

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

Leaving the option to do this by posting a GitHub Release is no issue to me.

If I have a slight preference for doing it via manual dispatch, that option is enabled as well (thank you!).

Overall, LGTM!

@confused-Techie
Copy link
Copy Markdown
Member Author

@DeeDeeG Sorry, if you wouldn't mind taking one last look, I've made some changes:

  • We manually trigger this workflow
  • It bumps the version according to what we specify, and will create a tag with that new version.
  • Then transpiles the package and creates another tag of that version.

This means we get complete automation, while leaving one tag for human consumption, and another for Pulsar consumption

Copy link
Copy Markdown
Member

@DeeDeeG DeeDeeG left a comment

Choose a reason for hiding this comment

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

Optional feedback about the automated commit message potentially including the version number, but it's exactly that, optional.

(Three proposals of how to address, only one (or none if not agreeing with the suggestion) needed, IDK why I went so far as to make three options, maybe just 'cause it's late lol.)

LGTM either way! Thank you for sticking through multiple rounds of review here, and for making this PR!

confused-Techie and others added 2 commits March 30, 2026 21:08
Co-authored-by: DeeDeeG <DeeDeeG@users.noreply.github.com>
Co-authored-by: DeeDeeG <DeeDeeG@users.noreply.github.com>
@confused-Techie
Copy link
Copy Markdown
Member Author

Thanks for all the review and getting this one past the finish line! Merging now

@confused-Techie confused-Techie merged commit fbfa249 into master Mar 31, 2026
0 of 4 checks passed
@confused-Techie confused-Techie deleted the auto-transpile branch March 31, 2026 04:12
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