ci: test both push and pull_request events #21

Merged
jalil merged 1 commit from push-wrtlnrrsrqoo into main 2025-07-11 19:26:04 +02:00
Owner

I think we have a bug with pull_request events so...

I think we have a bug with pull_request events so...
jalil self-assigned this 2025-07-10 19:40:26 +02:00
jalil added 1 commit 2025-07-10 19:40:26 +02:00
ci: test both push and pull_request events
Some checks failed
/ check (treefmt) (pull_request) Successful in 24s
/ report-size-push (pull_request) Failing after 1s
/ report-size-pr (pull_request) Failing after 2s
/ report-download-check-push (pull_request) Failing after 1s
/ report-download-check-pr (pull_request) Failing after 1s
c68e4deaad
I think we have a bug with pull_request events so...
jalil force-pushed push-wrtlnrrsrqoo from c68e4deaad to 70de79bafa 2025-07-10 19:47:34 +02:00 Compare
jalil force-pushed push-wrtlnrrsrqoo from 70de79bafa to 473a0f2852 2025-07-10 19:56:36 +02:00 Compare
jalil force-pushed push-wrtlnrrsrqoo from 473a0f2852 to 72e652f679 2025-07-10 20:03:39 +02:00 Compare
jalil force-pushed push-wrtlnrrsrqoo from 72e652f679 to 5f3b69c129 2025-07-10 20:06:22 +02:00 Compare
jalil force-pushed push-wrtlnrrsrqoo from 5f3b69c129 to f91d393cf9 2025-07-11 17:58:48 +02:00 Compare
jalil force-pushed push-wrtlnrrsrqoo from f91d393cf9 to 9ae615fc00 2025-07-11 19:10:26 +02:00 Compare

Flake output sizes

Definitions
  • Name: the name of the package/configuration.
  • Size: the closure size (size on disk/NAR size + all transitive dependencies).
  • NAR Size: the size of the build output (package without the dependencies).
  • [NAR] Size Change: the amount changed compared to the main branch.
Tips on reading this data
  • For NixOS/Home-Manager configurations you generally care only about the Size (closure size/size on disk).
    • Reduce the Size by disabling unneeded services/default packages.
  • For Packages you care about both the Size and the NAR Size.
    • Reduce the NAR Size by reducing the size of the build outputs, e.g. don't copy unnecessary data to the $out dir, optimize binaries for size, etc.
    • Reduce the Size by reducing the dependencies (e.g. buildInputs).
    • Don't worry too much about size, some dependencies are deduplicated, e.g. glibc adds ~40MiB to the Size, but is generally shared by ~every binary on the system, so, chances are, you are already including it from somewhere else and statically linking with e.g. musl is not gonna improve things.

Packages

Name Size Size Change NAR Size NAR Size Change
hello 32MiB 0B 268KiB 0B
<!-- AUTOGENERATED by nix-flake-outputs-size action --> ## Flake output sizes <details><summary><b>Definitions</b></summary> - `Name`: the name of the package/configuration. - `Size`: the closure size (size on disk/NAR size + all transitive dependencies). - `NAR Size`: the size of the build output (package without the dependencies). - `[NAR] Size Change`: the amount changed compared to the main branch. </details> <details><summary><b>Tips on reading this data</b></summary> - For NixOS/Home-Manager configurations you generally care only about the `Size` (closure size/size on disk). - Reduce the `Size` by disabling unneeded services/default packages. - For Packages you care about both the `Size` and the `NAR Size`. - Reduce the `NAR Size` by reducing the size of the build outputs, e.g. don't copy unnecessary data to the $out dir, optimize binaries for size, etc. - Reduce the `Size` by reducing the dependencies (e.g. `buildInputs`). - Don't worry too much about size, some dependencies are deduplicated, e.g. `glibc` adds ~40MiB to the `Size`, but is generally shared by ~every binary on the system, so, chances are, you are already including it from somewhere else and statically linking with e.g. `musl` is not gonna improve things. </details> ## Packages | Name | Size | Size Change | NAR Size | NAR Size Change | |------|-----:|------------:|---------:|----------------:| | `hello` | 32MiB | 0B | 268KiB | 0B |
jalil force-pushed push-wrtlnrrsrqoo from 9ae615fc00 to e5c9b8ab90 2025-07-11 19:16:16 +02:00 Compare
jalil force-pushed push-wrtlnrrsrqoo from e5c9b8ab90 to a492f64717 2025-07-11 19:19:15 +02:00 Compare
jalil force-pushed push-wrtlnrrsrqoo from a492f64717 to afce81f97f 2025-07-11 19:21:52 +02:00 Compare
jalil force-pushed push-wrtlnrrsrqoo from afce81f97f to 7d7a576f50 2025-07-11 19:25:03 +02:00 Compare
jalil merged commit 7d7a576f50 into main 2025-07-11 19:26:04 +02:00
jalil deleted branch push-wrtlnrrsrqoo 2025-07-11 19:26:04 +02:00
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: jalil/nix-flake-outputs-size#21
No description provided.