update-flake-lock icon indicating copy to clipboard operation
update-flake-lock copied to clipboard

feat: support signoff option

Open hackardoX opened this issue 5 months ago • 5 comments

Description

This PR add support for the signoff options supported by the Peter Evans's create-pull-request action

Checklist
  • [x] Tested functionality against a test repository (see "How to test changes")
  • [x] Added or updated relevant documentation (leave unchecked if not applicable)

hackardoX avatar Jul 09 '25 06:07 hackardoX

@cole-h Thank you for the review and for spotting the error in the readme 😄 Can you approve it now?

hackardoX avatar Jul 09 '25 21:07 hackardoX

Oh I see, I missed that!. What if we do remove the --commit-lock-file option from nix.ts, and we let the create-pull-request action take care of that?

hackardoX avatar Jul 09 '25 21:07 hackardoX

I played a bit with the action, and I finally make it work using create-pull-request action instead of using nix: image

https://github.com/andrea11/update-flake-lock-test/pull/3/commits

What do you think about this approach?

hackardoX avatar Jul 09 '25 22:07 hackardoX

Hi again @cole-h, I noticed that with my previous changes the flake update output went lost. I updated once more, to get something similar to what we had before: Before: image After: image

The output is not exactly the same as before, because I am using the output from nix flake update command rather and then delegate the commit to the github action create-pull-request.

Let me know if you consider this blocking

hackardoX avatar Jul 11 '25 12:07 hackardoX

@cole-h Any news on this PR? If it is too "complicated" or it has too many changes, let me know. I wanted to tackle this one before #200

hackardoX avatar Aug 11 '25 02:08 hackardoX