-
-
Notifications
You must be signed in to change notification settings - Fork 157
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
nixpkgs-fmt is now nixfmt #264
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I approve with the idea, but nixpkgs-fmt
should just be moved to the /DEPRECATED.md
list instead of deleted. Otherwise it looks good
https://github.com/nix-community/nixpkgs-fmt has been archived in favor of https://github.com/NixOS/nixfmt, we can delete the mention of the former to simplify life to newcomers. Note the organization repo transfer to NixOS too.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good now!
@@ -20,6 +20,10 @@ Here lie the following former awesome-list members as they have been archived, d | |||
|
|||
* [NixOS Weekly](https://weekly.nixos.org/) - *The* newsletter to stay informed about community updates. (Last update was made in 2021) | |||
|
|||
## Command-Line Tools | |||
|
|||
* [nixpkgs-fmt](https://github.com/nix-community/nixpkgs-fmt) formatter is now [nixfmt](https://github.com/NixOS/nixfmt). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That wording is kinda wrong. It has been replaced but it is not same thing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry. Want to follow-up with a fix? For me the most important part was to remove the duplication/confusion from the landing page; actual wording on DEPRECATED.md matters not very much to me.
https://github.com/nix-community/nixpkgs-fmt has been archived in favor of https://github.com/NixOS/nixfmt, we can delete the mention of the former to simplify life to newcomers.