Skip to content

chore: drop flake-utils - #377

Merged
sadjow merged 1 commit into
sadjow:mainfrom
ymelois:youn/push-xwyymrrvmurs
Aug 10, 2026
Merged

chore: drop flake-utils#377
sadjow merged 1 commit into
sadjow:mainfrom
ymelois:youn/push-xwyymrrvmurs

Conversation

@ymelois

@ymelois ymelois commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

flake-utils's lib.eachDefaultSystem can be replaced by a few line of simple nix code.
this makes it so we don't have to depend on yet another dependency.

kept the systems input for the extensibility that flake-utils introduced transitively.

flake-utils's lib.eachDefaultSystem can be replaced by a few line of
simple nix code. lib.eachDefaultSystem adds extra indirections and hides
standard flake features.

kept systems input

Signed-off-by: Youn Mélois <youn@melois.dev>
@ymelois
ymelois force-pushed the youn/push-xwyymrrvmurs branch from d712d1f to 0c400e4 Compare August 10, 2026 17:00
@ymelois

ymelois commented Aug 10, 2026

Copy link
Copy Markdown
Contributor Author

in my opinion d712d1f drifted too much from the initial definition, I may have been too quick on the judgement and pushed a fixed commit, which removed yours.

tell me if that's fine with you or if we should not write outputs with ${system} inline

@sadjow
sadjow merged commit e54a54c into sadjow:main Aug 10, 2026
4 checks passed
@sadjow

sadjow commented Aug 10, 2026

Copy link
Copy Markdown
Owner

Thanks for the contribution, and for revisiting the implementation. The inline ${system} approach is correct with the new recursiveUpdate helper, and I agree this version stays closer to the original change. Local validation and CI are green, and the PR is now merged. Much appreciated!

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.

2 participants