Skip to content

[#312] Add --follow-symlinks flag - #455

Merged
MattSturgeon merged 1 commit into
masterfrom
sereja313/add-follow-symlink
Sep 10, 2026
Merged

[#312] Add --follow-symlinks flag#455
MattSturgeon merged 1 commit into
masterfrom
sereja313/add-follow-symlink

Conversation

@Sereja313

@Sereja313 Sereja313 commented Sep 1, 2026

Copy link
Copy Markdown
Member

Formatting a symlink replaces it with a regular file. That stays the default and is now warned about, --follow-symlinks formats the file the link points at instead.

@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown

Nixpkgs diff

@jfly jfly left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

A couple comments for you, but LGTM. Thanks!

Comment thread README.md Outdated
Comment thread CHANGELOG.md Outdated
Formatting a symlink replaces it with a regular file. That stays the
default and is now warned about, --follow-symlinks formats the file
the link points at instead.

Fixes #312

Co-authored-by: Jeremy Fleischman <me@jfly.fyi>
Co-authored-by: Matt Sturgeon <matt@sturgeon.me.uk>
@MattSturgeon
MattSturgeon force-pushed the sereja313/add-follow-symlink branch from e41264f to 608248d Compare September 10, 2026 19:14

@MattSturgeon MattSturgeon left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I iterated on @jfly's README suggestion and replaced the CHANGELOG entry with a conventional commit (that Knope will pick up on).

@MattSturgeon
MattSturgeon merged commit 5861199 into master Sep 10, 2026
7 checks passed
@MattSturgeon
MattSturgeon deleted the sereja313/add-follow-symlink branch September 10, 2026 19:22
@nixfmt-ci nixfmt-ci Bot mentioned this pull request Sep 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants