Skip to content
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.

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

ls: upgrade uutils-term-grid to 0.5 #5485

Merged
merged 3 commits into from
May 1, 2024

Conversation

tertsdiepraam
Copy link
Member

Draft PR for now, because we need to release a new version of term-grid first, but I wanted to show how the new API changes in uutils/uutils-term-grid#27 are affecting ls.

@tertsdiepraam tertsdiepraam changed the title Adapt ls to upcoming version of uutils-term-grid ls: upgrade uutils-term-grid to 0.5 Feb 20, 2024
@sylvestre
Copy link
Contributor

@tertsdiepraam any plan here ? :)
thanks

@tertsdiepraam tertsdiepraam force-pushed the ls-new-term-grid branch 2 times, most recently from c84f7ed to e77d548 Compare April 30, 2024 21:03
@tertsdiepraam tertsdiepraam marked this pull request as ready for review April 30, 2024 21:04
Copy link

GNU testsuite comparison:

GNU test failed: tests/timeout/timeout. tests/timeout/timeout is passing on 'main'. Maybe you have to rebase?

Comment on lines 17 to 19
[dependencies]
ansi-width = "0.1.0"
clap = { workspace = true, features = ["env"] }
Copy link
Collaborator

Choose a reason for hiding this comment

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

The fact that every entry except this one says workspace = true and doesn't provide a specific version number makes me wonder why it's the other way around here.

Copy link
Member Author

Choose a reason for hiding this comment

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

Because I forgot 😅

src/uu/ls/Cargo.toml Outdated Show resolved Hide resolved
Copy link

github-actions bot commented May 1, 2024

GNU testsuite comparison:

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

@tertsdiepraam
Copy link
Member Author

Ah I was just gonna do that. Thanks :)

@sylvestre sylvestre merged commit 3346b41 into uutils:main May 1, 2024
67 of 68 checks passed
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.

3 participants