2
2
Fork
You've already forked nix
2

helix: use upstream repo #5

Manually merged
koibtw merged 1 commit from aquifolly/evergarden-nix:push-uysrnvpwvqrq into main 2026年07月12日 15:55:50 +02:00
Contributor
Copy link

helix is already in the sources, but it isn't used for some reason

helix is already in the sources, but it isn't used for some reason

hiii:3

thnx for the pr ~this was originally added by helix port maintainer so i didnt notice the duplicate code lol

a few points:

  • can you save the "evergarden_${cfg.variant}_${cfg.accent}" string in a var (called name) and reuse that.
  • can you use "${ports.helix}/subpath"; thats what we do in the other modules (and i really dont like the nix + operator:3
hiii:3 thnx for the pr ~this was originally added by helix port maintainer so i didnt notice the duplicate code lol a few points: - can you save the `"evergarden_${cfg.variant}_${cfg.accent}"` string in a var (called name) and reuse that. - can you use `"${ports.helix}/subpath"`; thats what we do in the other modules (and i really dont like the nix `+` operator:3
koibtw left a comment
Copy link

lgtm :3

lgtm :3
koibtw referenced this pull request from a commit 2026年07月12日 15:55:40 +02:00
koibtw manually merged commit c2d043a96f into main 2026年07月12日 15:55:50 +02:00
Sign in to join this conversation.
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
evergarden/nix!5
Reference in a new issue
evergarden/nix
No description provided.
Delete branch "aquifolly/evergarden-nix:push-uysrnvpwvqrq"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?