Skip to content

fix(nix): no longer trying to access non existent attribute#73

Closed
Libadoxon wants to merge 1 commit intojavalsai:masterfrom
Libadoxon:master
Closed

fix(nix): no longer trying to access non existent attribute#73
Libadoxon wants to merge 1 commit intojavalsai:masterfrom
Libadoxon:master

Conversation

@Libadoxon
Copy link

fixes the nix build. cfg-file doesn't exist if get-cfg is null, so checking if get-cfg is null instead of cfg-file

@Libadoxon
Copy link
Author

Libadoxon commented Aug 17, 2025

Okay I just took a closer look at the nix code, with your permission I would rewrite the whole thing (mostly). Nothing against your nix skills but there a some serious anti patters and maintainability burdens that can be removed.

@grialion
Copy link
Collaborator

Okay I just took a closer look at the nix code, with your permission I would rewrite the whole thing (mostly). Nothing against your nix skills but there a some serious anti patters and maintainability burdens that can be removed.

It'd be really neat if you would do that!

@Libadoxon
Copy link
Author

Superseded by #74

@Libadoxon Libadoxon closed this Aug 17, 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.

2 participants

Comments