pkgjs / action

Github Actions tooling for testing Node.js packages
MIT License
12 stars 2 forks source link

Support for release workflow #65

Open wesleytodd opened 11 months ago

wesleytodd commented 11 months ago

I would love to see a managed release workflow here as well. I see two ways we could go:

  1. Release Please
  2. Call version and release package.json scripts

I personally like the first option a lot, but I know that is more opinionated. Maybe we could even have both?

ljharb commented 11 months ago

We shouldn’t use a single factor approach, but if we can get a two factor one that sounds great.

wesleytodd commented 11 months ago

As much as I agree, there is not one today which can do this. We have discussed this in a few different forums, but I don't see any solutions here other than author local machine, and that is just not what users want nor is it was is best for projects other than ones with authors like you who are deeply invested in the ecosystem.

ljharb commented 11 months ago

There is if you use an external service, like Step Security's "wait for secrets" mechanism.

wesleytodd commented 11 months ago

Got a link? Haven't seen that one yet.

ljharb commented 11 months ago

https://github.com/step-security/wait-for-secrets - i use it on eslint-plugin-react. There's also https://github.com/GoogleCloudPlatform/wombat-dressing-room, but that requires you to deploy something yourself.

wesleytodd commented 11 months ago

Yeah wombat dressing room was the one I knew about. I will check out wait-for-secrets. I am happy if we can get a good solution here, so if that is it I am onboard.

dominykas commented 11 months ago

This does not publish anything to npm - does 2FA even apply?

wesleytodd commented 11 months ago

My proposal was that it would also have a workflow which did release. I was thinking it could be separate, but since this is just called action I assumed it might make sense for it to contain more than just the test setup.

dominykas commented 11 months ago

workflow which did release [..] more than just the test setup.

Sorry, not sure what you mean?

wesleytodd commented 11 months ago

I am proposing we add a new workflow for release/publish.

dominykas commented 11 months ago

Yeah, that's fine, but in the context of 2FA - do we need anything? This repo is not publishing anything on npm, and internal Github stuff can be handled by the token, or am I missing something?

wesleytodd commented 11 months ago

I think @ljharb was pointing out that providing a release workflow to other repos without 2FA was an issue.

dominykas commented 11 months ago

Oh, OK, yeah, that I agree on. I assumed this was about having a release workflow in here, but the request here is about providing a workflow for others to use, right?

ljharb commented 11 months ago

Yes, that was my understanding.

wesleytodd commented 11 months ago

Yep, that was my thought, a workflow for others to use. I was just trying to think of things that would get more folks setup without having to do it all on their own.