Skip to content

treewide: run nix fmt#362

Open
stepbrobd wants to merge 4 commits into
nix-community:mainfrom
stepbrobd:format
Open

treewide: run nix fmt#362
stepbrobd wants to merge 4 commits into
nix-community:mainfrom
stepbrobd:format

Conversation

@stepbrobd

Copy link
Copy Markdown
Member

need rebase on #358 and add the commit to blame ignore file

@stepbrobd
stepbrobd marked this pull request as ready for review July 7, 2026 08:49
@stepbrobd
stepbrobd requested a review from NickCao July 7, 2026 21:32
Comment thread formatter.nix
taplo,
}:

writeShellScriptBin "formatter" ''

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

im a bit hesitant on this because in a future PR i would want to drop flake inputs that are not nixpkgs/nixpkgs-stable

maybe we could do what nixvim (and a few other projects) is doing splitting the dev inputs vs actual inputs exposed to users but this also feels unnecessary

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

treefmt is available in nixpkgs, we can just have it.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

didn't know this 😳 i'll try this in a few days

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.

2 participants