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

doc/nix fmt: Do not document behavior we do not control #11613

Merged
merged 1 commit into from
Sep 29, 2024

Conversation

roberth
Copy link
Member

@roberth roberth commented Sep 29, 2024

Motivation

Context

Priorities and Process

Add 👍 to pull requests you find important.

The Nix maintainer team uses a GitHub project board to schedule and track reviews.

Copy link
Contributor

@jfly jfly left a comment

Choose a reason for hiding this comment

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

This is much improved, thank you!

This leads to confusion about what the command does.
E.g. #9359

- Move the description up

- Remove details about the individual formatters
@github-actions github-actions bot added the with-tests Issues related to testing. PRs with tests have some priority label Sep 29, 2024
@fricklerhandwerk fricklerhandwerk merged commit 7913818 into master Sep 29, 2024
18 checks passed
@fricklerhandwerk fricklerhandwerk deleted the nix-fmt-doc branch September 29, 2024 18:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation new-cli Relating to the "nix" command with-tests Issues related to testing. PRs with tests have some priority
Projects
None yet
Development

Successfully merging this pull request may close these issues.

nix fmt formats files in .git, leading to a corrupted repository
3 participants