Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Verify in CI: small runtime dependencies #393

Closed
srid opened this issue Dec 6, 2022 · 1 comment
Closed

Verify in CI: small runtime dependencies #393

srid opened this issue Dec 6, 2022 · 1 comment
Assignees
Labels
good first issue Good for newcomers Works only on Nix This feature will work only on Nix.

Comments

@srid
Copy link
Owner

srid commented Dec 6, 2022

Test in CI that the emanote closure size doesn't bloat up again (#295)

Current:

❯ nix path-info --json -S .#default | jq 'map(select(.closureSize > 0.4e9)) | sort_by(.closureSize) | map([.path, .closureSize])'
warning: Using saved setting for 'extra-substituters = https://cache.garnix.io' from ~/.local/share/nix/trusted-settings.json.
warning: Using saved setting for 'extra-trusted-public-keys = cache.garnix.io:CTFPyKSLcx5RMJKfLo5EEPUObbA78b0YQ2DTCJXqr9g=' from ~/.local/share/nix/trusted-settings.json.
[
  [
    "/nix/store/3h9p5bi32263wqdnpf63f1rw0c722fln-emanote-1.0.1.0",
    542715032
  ]
]

Potential test: ensure that the following command ("Show every path whose closure is bigger than 1 GB, sorted by closure size:") returns empty results:

❯ nix path-info --json -S .#default | jq 'map(select(.closureSize > 1e9)) | sort_by(.closureSize) | map([.path, .closureSize])'  
warning: Using saved setting for 'extra-substituters = https://cache.garnix.io' from ~/.local/share/nix/trusted-settings.json.
warning: Using saved setting for 'extra-trusted-public-keys = cache.garnix.io:CTFPyKSLcx5RMJKfLo5EEPUObbA78b0YQ2DTCJXqr9g=' from ~/.local/share/nix/trusted-settings.json.
[]
@srid srid added good first issue Good for newcomers Works only on Nix This feature will work only on Nix. labels Dec 6, 2022
@srid srid changed the title Verify small binary size in CI Verify in CI: small runtime dependencies Dec 6, 2022
@srid srid self-assigned this Dec 10, 2023
@srid
Copy link
Owner Author

srid commented Dec 10, 2023

Added the check to Jenkins: #475

@srid srid closed this as completed Dec 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers Works only on Nix This feature will work only on Nix.
Projects
None yet
Development

No branches or pull requests

1 participant