Skip to content

ls: fix quoting alignment, add tests for default quoting style in TTY and quoting alignment#6402

Merged
sylvestre merged 3 commits intouutils:mainfrom
RenjiSann:main
May 16, 2024
Merged

ls: fix quoting alignment, add tests for default quoting style in TTY and quoting alignment#6402
sylvestre merged 3 commits intouutils:mainfrom
RenjiSann:main

Conversation

@RenjiSann
Copy link
Collaborator

Fixes #6386

@RenjiSann RenjiSann force-pushed the main branch 2 times, most recently from faa1d3d to c52e852 Compare May 14, 2024 19:18
@github-actions
Copy link

GNU testsuite comparison:

Skipping an intermittent issue tests/tail/inotify-dir-recreate (passes in this run but fails in the 'main' branch)

@sylvestre
Copy link
Contributor

I guess you saw that some jobs are failing

@RenjiSann
Copy link
Collaborator Author

I guess you saw that some jobs are failing

Yes, sorry for that. I always forget about linting and windows stuff.

@github-actions
Copy link

GNU testsuite comparison:

Skip an intermittent issue tests/tail/inotify-dir-recreate (fails in this run but passes in the 'main' branch)

@RenjiSann
Copy link
Collaborator Author

Forgot to add #[cfg(unix)] on all TTY tests.

@github-actions
Copy link

GNU testsuite comparison:

Skipping an intermittent issue tests/tail/inotify-dir-recreate (passes in this run but fails in the 'main' branch)

@RenjiSann
Copy link
Collaborator Author

@sylvestre if you don't mind taking look. I don't understand why is the macos test failing.

@sylvestre
Copy link
Contributor

@RenjiSann it isn't your fault. it is unrelated

@sylvestre sylvestre merged commit 0e5614d into uutils:main May 16, 2024
@sylvestre
Copy link
Contributor

LGTM, thanks

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.

ls: Unexpected space before stdout in TTY output

2 participants