Skip to content

Conversation

@cakebaker
Copy link
Contributor

While reviewing #7449 I noticed the following snippet:

if !target_dir.is_dir() {
    return Err(LnError::TargetIsDirectory(target_dir.to_owned()).into());
}

This PR fixes the snippet by renaming the error variant to TargetIsNotADirectory.

from TargetIsDirectory to TargetIsNotADirectory
@github-actions
Copy link

GNU testsuite comparison:

Skipping an intermittent issue tests/timeout/timeout (passes in this run but fails in the 'main' branch)

@sylvestre sylvestre merged commit f2579e5 into uutils:main Jun 1, 2025
113 of 114 checks passed
@cakebaker cakebaker deleted the ln_rename_error_variant branch June 2, 2025 05:01
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.

2 participants