Skip to content

Print when a new file is linked or copied - #310

Open
9999years wants to merge 1 commit into
thoughtbot:mainfrom
9999years:print-new-link
Open

Print when a new file is linked or copied#310
9999years wants to merge 1 commit into
thoughtbot:mainfrom
9999years:print-new-link

Conversation

@9999years

Copy link
Copy Markdown

When I use rcup, I get no output, even if a new file is linked:

$ rcup
$

When I use rcup -v, I get a list of every linked file in my dotfiles (boring):

$ rcup -v
identical /Users/9999years/.cargo/config.toml
identical /Users/9999years/.config/1Password/ssh/agent.toml
identical /Users/9999years/.config/alacritty/alacritty.yml
...

This displays when new files are linked at all log levels. (Maybe this should be hidden behind an rcrc option?)

@9999years
9999years requested a review from DoodlingDev as a code owner August 22, 2025 18:23
When I use `rcup`, I get no output, even if a new file is linked:

```
$ rcup
$
```

When I use `rcup -v`, I get a list of every linked file in my dotfiles
(boring):

```
$ rcup -v
identical /Users/9999years/.cargo/config.toml
identical /Users/9999years/.config/1Password/ssh/agent.toml
identical /Users/9999years/.config/alacritty/alacritty.yml
...
```

This displays when new files are linked at all log levels:

```
$ rcup
ln ~/.dotfiles/my-new-dotfile ~/.my-new-dotfile
```

Is this OK, or does it conflict with the minimal output of `rcup`? Maybe
this should be hidden behind an `rcrc` option?
fictionic added a commit to fictionic/rcm-fast that referenced this pull request Aug 15, 2026
this is an adjusted version of the change proposed by Rebecca Turner in
upstream PR thoughtbot#310. the differences are:
- use the same print format as -v
- don't double-print when -v is passed
- update affected tests, as this changes behavior

Suggested-by: Rebecca Turner <[email protected]>
Link: thoughtbot#310
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.

1 participant