Skip to content

Conversation

@fingolfin
Copy link
Member

@fingolfin fingolfin commented Sep 13, 2025

This takes some of #38 into account by reordering and rephrasing some stuff, dropping the minimal example etc.

But not everything is done yet, e.g. the suggestions for step 5 are not taken into account -- partially because I'd like to do this while actually making a release -- e.g. as I recall, there is a delay before one "sees" any progress bars etc., unless one manually reloads, and if that recollection is right, this should also be described... in any case, the text must match what really happens so I don't want to write it blindly right now.

But partial progress is still progress, right? ;-)

Preview at https://github.com/gap-actions/release-pkg/blob/mh/README/README.md

@cdwensley
Copy link

The latest version of the instructions look fine.
However, I still do not understand how to achieve a dry run.
My guess is to edit the release.yml file, changing the "default: false" under "dry-run" to "default: true"
before starting the action process. Is that correct - if so, perhaps make that clearer?

@stertooy
Copy link
Contributor

@cdwensley

Borrowing Max's image from the README: clicking the first checkbox in this menu before clicking 'Run workflow' should do a dry run.

herp

I am planning to add instructions for this in the README as part of #23, but I haven't got around to it yet.

@cdwensley
Copy link

Thanks. I suppose that should have been obvious, but the first box could have been: "dry-run: only create ..."

@fingolfin
Copy link
Member Author

We can improve this further in a follow-up PRs

@fingolfin fingolfin enabled auto-merge (squash) September 14, 2025 23:41
@fingolfin fingolfin disabled auto-merge September 22, 2025 21:47
@fingolfin fingolfin merged commit 244c3b3 into main Sep 22, 2025
3 checks passed
@fingolfin fingolfin deleted the mh/README branch September 22, 2025 21:47
@stertooy stertooy mentioned this pull request Oct 31, 2025
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.

4 participants