Skip to content

feat(lib): allow multiple files - #145

Open
yajo wants to merge 1 commit into
numtide:mainfrom
moduon:lib-extended
Open

yajo wants to merge 1 commit into
numtide:mainfrom
moduon:lib-extended

Conversation

@yajo

@yajo yajo commented Feb 17, 2026

Copy link
Copy Markdown
Contributor

This makes handling libs more comfortable.

Also adds docs for lib-tests automatic check.

# lib/strings.nix - Attrset of functions
{ inputs, flake, ... }:
{
capitalize = str: /* ... */;

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what happens if you want to access lib.greet from this file?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You should find it in flake.lib.greet.

@yajo

yajo commented Mar 9, 2026

Copy link
Copy Markdown
Contributor Author

Any progress here? Thanks.

@yajo

yajo commented Mar 26, 2026

Copy link
Copy Markdown
Contributor Author

I have flakes where lib keeps growing and I'd really appreciate if you could take a look at this and push it forward. Thanks!

@yajo

yajo commented May 7, 2026

Copy link
Copy Markdown
Contributor Author

Hello! Any review here please? 😅

This makes handling libs more comfortable.

Also adds docs for lib-tests automatic check.
@yajo

yajo commented May 19, 2026

Copy link
Copy Markdown
Contributor Author

I decided to simplify the implementation a lot.

Instead of auto-detecting if specialArgs should be passed or not, they are now passed always. This makes it easier to reason about the code, and reduces the implementation to 3 lines.

I hope this helps the PR getting reviewed and merged.

@yajo

yajo commented Jul 28, 2026

Copy link
Copy Markdown
Contributor Author

If I send you a picture of a cute cat will you review this?

Sandwich cat

This branch has not been deployed

No deployments
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