Skip to content

🚚 Massive update: New features, Node 20, much less dependencies, bug fixes, more tests#384

Closed
mountainash wants to merge 109 commits into
BetaHuhn:masterfrom
mountainash:main
Closed

🚚 Massive update: New features, Node 20, much less dependencies, bug fixes, more tests#384
mountainash wants to merge 109 commits into
BetaHuhn:masterfrom
mountainash:main

Conversation

@mountainash
Copy link
Copy Markdown

@mountainash mountainash commented Apr 12, 2024

  • FEATURE ✨ Added --archive=tgz arg to Vercel deploy when using PREBUILT option to allow built deployments over 1500 files

  • Added defaults

  • Added GitHub Actions workflow to test the action

  • GITHUB_REPOSITORY is no longer a required input (it can be deduced using the GITHUB_TOKEN)

  • Allow Vercel domain aliases to be set on PRs

  • Using @actions/exec instead of spawn = require('child_process') for better OS portability

  • Fix: EditorConfig conflicted with Eslint rules

  • Using npx to run Vercel CLI to avoid version miss-matches (as seen in Error: undefinedVercel CLI 31.2.2 #374, Error: undefinedVercel CLI 31.0.1 #367, Error: undefinedVercel CLI 28.0.1 #226)

  • FEATURE ✨ Add a Workflow Summary to each run

  • Exporting VERCEL_PREVIEW_URL and VERCEL_DEPLOYMENT_UNIQUE_URL for use in other job steps

  • Using @actions/core to correctly get boolean and multilined inputs

  • Removed default "deploy" label from PRs

  • More emojis in logs 📝

  • Better accessibility formatting for comment table

  • remove references of [skip ci], it's deprecated

  • Replace _ in alias domains with - to match allowed domain names

  • FEATURE ✨ Transfer runtime secrets/envars from GHAction to Vercel Settings

  • See CHANGELOG for many 📦 dependencies updates (inc. Node 20) and bug fixes

👷‍♂️ Added Debug Action for in project testing
@BetaHuhn
Copy link
Copy Markdown
Owner

BetaHuhn commented May 5, 2024

sorry I didn't break this up in to many PR's - but that comes with quite the "admin" overhead that I wasn't ready to make on what looked to be an abandoned project (I recently did many PR's on another project which were ignored and autoclosed shortly after - it's quite disheartening to see efforts wasted 😿).

Totally understand! I have had similar experiences before 😅

If you don't have the time and are looking to offload this project, I'd gladly become the new care-taker for it - I even have a few more small bugs to fix and reliability features that will help make the project even more robust.

Nice! I had some ideas/feature roadmap a while back as well, have to dig up my notes sometime.

How would a transfer work? Aren't everyone's actions.yml files hardcoded to the repo on your GitHub user account name?

Yes, all current users reference the action directly via betahuhn/deploy-to-vercel-action which means a transfer of the repository would automatically change the source of the action which carries with it some security considerations.

Personally I think a change in ownership of a project like this should be communicated well and be done in a transparent way with a explicit action having to be taken by the user to change to the new owner.

The simplest approach I can think off is to create one last release that mentions the change of maintainers, add a big banner pointing to your fork so everyone can just switch over at their own choosing and deprecate the action (sadly not possible on GitHub right now) / archive the repository.

Less aggressive approach would be to just advertise your fork as a improved version and keep this one around.

@mountainash
Copy link
Copy Markdown
Author

mountainash commented May 5, 2024

All choices work for me... I made the changes for me and my use of your Action at @ginetta - but it's open-source and anyone is already free to use mountainash/fork-deploy-to-vercel-action@develop - but if you/we go the "offical" take-over path, maybe I should rename mine to just mountainash/deploy-to-vercel-action (removing "fork") to make the change over easier for users.

There's a chance some of the changes I've made to update the GitHub Actions core package could effect some projects depending on how they are built. I've only really heavily tested using the "PREBUILT" flow. I think it would be good to release my changes as v2.

I'd also have to release it onto the GitHub Actions Marketplace, but I think it could be confusing if there's two with the same name there, so how would you suggest to handle this?

TODO

@pm0u
Copy link
Copy Markdown

pm0u commented May 23, 2024

Is this still planning to be released in another repo? I really would like to see the --archive=tgz option exposed without prebuilt. Happy to contribute if development is moving elsewhere

@mountainash mountainash closed this Mar 8, 2025
@mountainash mountainash deleted the main branch March 8, 2025 11:42
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.

5 participants