andir / npins

Nix dependency pinning. Very similar to Niv but has a few features that I personally wanted.
European Union Public License 1.2
198 stars 16 forks source link

Add support for Forgejo #95

Closed crop2000 closed 1 month ago

piegamesde commented 2 months ago

Thank you, that looks like a pretty good start already. How does forgejo differ from Gitea? Could we maybe even support both simply by adding an alias to the CLI subcommand?

Also, please add some tests, both a Rust test (I'd suggest using the Lix repo as test subject) and a mock test in test.nix

P.S.: Your commit is credited to @olafklingt instead of you, please double-check that this is indeed correct

crop2000 commented 2 months ago

i did hand test it with lix, and can write some Rust test and a mock tests. I don't know about the differences to gittea. But as what we are using is fairly similar to github i don't expect that they diverged on this topic.

crop2000 commented 2 months ago

I tend to feel that adding gitea support via a alias is just cluttering the cli ux ... so i didn't add an alias.

crop2000 commented 1 month ago

are you interested in this pr?

crop2000 commented 1 month ago

:)