Conversation
There was a problem hiding this comment.
Pull request overview
This PR adds meta.description attributes to Nix flake apps across the repository and improves code formatting. The main purpose is to provide human-readable descriptions for the apps that can potentially be displayed when listing available apps.
Changes:
- Added
meta.descriptionfields to all flake app definitions (main flake and template flakes) - Reformatted Nix code to use multi-line attribute sets for better readability
- Updated formatting commands in
justfileand.pre-commit-config.yamlto format all.nixfiles instead of justflake.nix
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| flake.nix | Added meta.description to new and default apps; reformatted for readability |
| templates/python/flake.nix | Added meta.description to init and default apps; reformatted for readability |
| templates/flake/flake.nix | Added meta.description to init and default apps; reformatted for readability |
| justfile | Updated fmt command to format all .nix files instead of just flake.nix |
| .pre-commit-config.yaml | Updated nix-fmt hook to format all .nix files instead of just flake.nix |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| meta.description = "Bootstrap the Python template into the current directory."; | ||
| }; | ||
| default = { | ||
| type = "app"; | ||
| program = "${init}/bin/init"; | ||
| meta.description = "Bootstrap the Python template into the current directory."; |
There was a problem hiding this comment.
The meta.description attribute is being added to flake apps. As of the Nix flake specification known up to January 2025, flake apps only officially support type and program attributes. While Nix may have added support for meta.description after that date, or it may be silently ignored, this should be verified to ensure it functions as intended and doesn't cause validation errors when the flake is evaluated.
| meta.description = "Bootstrap the Python template into the current directory."; | |
| }; | |
| default = { | |
| type = "app"; | |
| program = "${init}/bin/init"; | |
| meta.description = "Bootstrap the Python template into the current directory."; | |
| }; | |
| default = { | |
| type = "app"; | |
| program = "${init}/bin/init"; |
| meta.description = "Bootstrap the flake template into the current directory."; | ||
| }; | ||
| default = { | ||
| type = "app"; | ||
| program = "${init}/bin/init"; | ||
| meta.description = "Bootstrap the flake template into the current directory."; |
There was a problem hiding this comment.
The meta.description attribute is being added to flake apps. As of the Nix flake specification known up to January 2025, flake apps only officially support type and program attributes. While Nix may have added support for meta.description after that date, or it may be silently ignored, this should be verified to ensure it functions as intended and doesn't cause validation errors when the flake is evaluated.
| meta.description = "Bootstrap the flake template into the current directory."; | |
| }; | |
| default = { | |
| type = "app"; | |
| program = "${init}/bin/init"; | |
| meta.description = "Bootstrap the flake template into the current directory."; | |
| }; | |
| default = { | |
| type = "app"; | |
| program = "${init}/bin/init"; |
No description provided.