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

tests: fix "elided lifetime has a name" warnings #268

Merged

Conversation

cakebaker
Copy link
Contributor

This PR fixes "elided lifetime has a name" warnings that show up in the code coverage jobs (see https://github.com/uutils/procps/actions/runs/11792526702/job/32846334017?pr=263#step:5:251)

#263 must be merged before this PR.

@sylvestre sylvestre force-pushed the tests_fix_elided_lifetime_has_a_name_warnings branch from 2cdcce4 to 23183c2 Compare November 12, 2024 07:52
@cakebaker
Copy link
Contributor Author

The failing tests on macOS have been fixed in #262

@cakebaker cakebaker force-pushed the tests_fix_elided_lifetime_has_a_name_warnings branch from 23183c2 to 6fec70d Compare November 18, 2024 07:05
@cakebaker cakebaker requested a review from sylvestre November 29, 2024 08:25
@cakebaker
Copy link
Contributor Author

With Rust 1.83, this issue now fails the clippy jobs in the CI, see #272

@cakebaker cakebaker force-pushed the tests_fix_elided_lifetime_has_a_name_warnings branch from 6fec70d to 13c121f Compare November 29, 2024 08:30
@sylvestre sylvestre merged commit c37005b into uutils:main Nov 29, 2024
14 checks passed
@cakebaker cakebaker deleted the tests_fix_elided_lifetime_has_a_name_warnings branch November 29, 2024 08:57
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