Skip to content

Conversation

@BirdeeHub
Copy link
Owner

@BirdeeHub BirdeeHub commented Feb 3, 2026

to run the tests, use nix flake check -Lv ./ci

to build the docs, use nix run ./ci

This also tests that the docs worked as part of the checks now.

wlib.checks now contains wlib.checks.helper.* and wlib.checks.wrapper.* however I am quite confident nobody outside of this repository is interacting with wlib.checks as it is not documented and does nothing useful for users other than being ran in github actions to catch bugs.

@BirdeeHub BirdeeHub changed the title ci(checks-docs): moved tests to docs flake. renamed docs -> ci ci(checks-docs): moved tests to ci flake. renamed docs/ -> ci/ Feb 3, 2026
@BirdeeHub
Copy link
Owner Author

BirdeeHub commented Feb 3, 2026

oh interesting it can't pull even the name for the docs if the system doesn't match...

I feel like there is probably a real fix for that.

Obviously the module isnt supposed to work on that system, so the fix should be something that makes the docgen not try to fill in the default? Or, the name is there anyway and we aren't building it, so maybe allowUnsupportedSystem for the docs?

I should also add supported meta.platforms to the docs for the wrapper modules maybe? But that will be a second PR.

@BirdeeHub BirdeeHub force-pushed the move_ci branch 9 times, most recently from aa03420 to 2f05ce3 Compare February 3, 2026 02:57
@BirdeeHub
Copy link
Owner Author

BirdeeHub commented Feb 3, 2026

Also, if the cache action doesnt work as well as hoped and makes it always take 20 minutes then I will be removing it again, but I'm going to give it another chance.

Because the tests themselves take more like 1 minute on the runner, so it is barely worth it at the moment.

Edit: nope. Even when it works and only updates 1 thing it is still slower to upload the delta than just running all the tests again currently. We don't really "build" stuff here, other than some config files and wrapper scripts so everything just comes from the nixpkgs cache.

to run the tests, use `nix flake check -Lv ./ci`
@BirdeeHub BirdeeHub merged commit 5c01bb9 into main Feb 3, 2026
2 checks passed
@BirdeeHub BirdeeHub deleted the move_ci branch February 3, 2026 03:35
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.

1 participant