8000 `uniq`: remove `strum` dependency by tertsdiepraam · Pull Request #4629 · uutils/coreutils · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

uniq: remove strum dependency #4629

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
Mar 25, 2023
Merged

uniq: remove strum dependency #4629

merged 1 commit into from
Mar 25, 2023

Conversation

tertsdiepraam
Copy link
Member

The strum dependency was only used here and it wasn't doing a lot. It also pulled in two transitive dependencies.

@tertsdiepraam tertsdiepraam changed the title uniq: convert string to Delimiters manually instead of via strum uniq: remove strum dependency Mar 25, 2023
@tertsdiepraam tertsdiepraam mentioned this pull request Mar 25, 2023
Closed
1 task
@sylvestre
Copy link
Contributor

Not doing much indeed

@sylvestre sylvestre merged commit c0bd851 into uutils:main Mar 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milest 3FD2 one
No milestone
Development

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