foundry.nix
foundry.nix copied to clipboard
Add integration test for PRs
Too easy to break the build when merging fixes, e.g. #10 -> #11
Currently, you have tiny inputs for the flake, and it's good that someone adds your flake as an input. What do you think about adding some other dependencies but only for CI?
Seems like they are unnecessary for the flake, so we shouldn't include any additional dependencies to the main flake. Should we create another flake only for CI? Or just use CI without nix magic (formatting, testing and so on)?
@selfuryon I don't have strong feelings here, the bundle itself is comparatively large (50MB+) that I don't feel bad adding a few kb of dependencies.
A separate flake in a subdirectory just for testing could work too though, if it's convenient. :)
Overall, any amount of integration testing is better than nothing, so any approach is appreciated!
Got that, I will try to do that this week :)