Skip to content

ls: implement --hyperlink #5622

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Dec 7, 2023
Merged

ls: implement --hyperlink #5622

merged 1 commit into from
Dec 7, 2023

Conversation

cakebaker
Copy link
Contributor

This PR implements --hyperlink. Not included in this PR is the encoding of special characters and thus the hyperlink GNU test should still fail.

sylvestre
sylvestre reviewed Dec 6, 2023
View reviewed changes
@cakebaker cakebaker force-pushed the ls_hyperlink branch 2 times, most recently from c947608 to 6fe227f Compare December 6, 2023 16:24
@cakebaker cakebaker marked this pull request as draft December 6, 2023 16:35
@cakebaker cakebaker marked this pull request as ready for review December 7, 2023 07:16
@github-actions GitHub Actions
Copy link

github-actions bot commented Dec 7, 2023

GNU testsuite comparison:

Congrats! The gnu test tests/rm/rm2 is no longer failing!
Congrats! The gnu test tests/tail/inotify-dir-recreate is no longer failing!
Skip an intermittent issue tests/rm/rm1

@sylvestre sylvestre merged commit d63d4a7 into uutils:main Dec 7, 2023
@cakebaker cakebaker deleted the ls_hyperlink branch December 7, 2023 08:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sylvestre sylvestre sylvestre left review comments

Assignees
No one assigned
Labels
None yet
Projects
GNU Compatibility
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@cakebaker @sylvestre