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.

Sign up for GitHub

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

Jump to bottom

The default nix flake installable is heavy #160

Closed
TristanCacqueray opened this issue May 3, 2022 · 1 comment · Fixed by #161
Closed

The default nix flake installable is heavy #160

TristanCacqueray opened this issue May 3, 2022 · 1 comment · Fixed by #161
Labels
enhancement New feature or request

Comments

@TristanCacqueray
Copy link
Contributor

Is your feature request related to a problem? Please describe.

Running nix run github:numtide/treefmt pulls about 5GB of dependencies.
It seems like the linters are somehow downloaded even when they are not used.

Describe the solution you'd like

The default app should be lightweight and downloading the linters tools should be optional.
@TristanCacqueray TristanCacqueray added the enhancement New feature or request label May 3, 2022
TristanCacqueray added a commit to TristanCacqueray/treefmt that referenced this issue May 3, 2022
@TristanCacqueray
nix: move ci dependencies to the devShell
4dfda7d 
Fixes numtide#160
@TristanCacqueray TristanCacqueray mentioned this issue May 3, 2022
Merged
@TristanCacqueray
Copy link
Contributor Author

With #161, running nix run github:numtide/treefmt/4dfda7d263b752527351dd6459aee7600fe63b4f now only pulls the rust toolchain, which is about 1.6GB.

TristanCacqueray added a commit to TristanCacqueray/treefmt that referenced this issue May 11, 2022
@TristanCacqueray
nix: move ci dependencies to the devShell
47bd12a 
Fixes numtide#160
zimbatm pushed a commit that referenced this issue May 12, 2022
@TristanCacqueray
nix: move ci dependencies to the devShell (#161)
be1d854 
Fixes #160
brianmcgee pushed a commit to brianmcgee/treefmt that referenced this issue Dec 9, 2022
@TristanCacqueray @brianmcgee
nix: move ci dependencies to the devShell (numtide#161)
b781479 
Fixes numtide#160
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

nix: move ci dependencies to the devShell TristanCacqueray/treefmt
1 participant
@TristanCacqueray