Skip to content

Enable some pedantic rules#8203

Merged
RenjiSann merged 3 commits intouutils:mainfrom
gabelluardo:add-clippy-rules
Jul 17, 2025
Merged

Enable some pedantic rules#8203
RenjiSann merged 3 commits intouutils:mainfrom
gabelluardo:add-clippy-rules

Conversation

@gabelluardo
Copy link
Contributor

Ref #4949 #8201

@github-actions
Copy link

GNU testsuite comparison:

Skip an intermittent issue tests/misc/tee (fails in this run but passes in the 'main' branch)
Skip an intermittent issue tests/timeout/timeout (fails in this run but passes in the 'main' branch)

@github-actions
Copy link

GNU testsuite comparison:

Skip an intermittent issue tests/misc/stdbuf (fails in this run but passes in the 'main' branch)

@github-actions
Copy link

GNU testsuite comparison:

Skipping an intermittent issue tests/misc/stdbuf (passes in this run but fails in the 'main' branch)

@gabelluardo gabelluardo force-pushed the add-clippy-rules branch 2 times, most recently from 83aa296 to 9d862cc Compare July 16, 2025 17:14
@gabelluardo
Copy link
Contributor Author

What's our policy for resolving merge conflicts? Should we use merge or rebase here?

@RenjiSann
Copy link
Collaborator

What's our policy for resolving merge conflicts? Should we use merge or rebase here?

I'm personnally all in for rebases

@gabelluardo gabelluardo force-pushed the add-clippy-rules branch 2 times, most recently from eef9202 to c5ffe91 Compare July 17, 2025 07:20
@github-actions
Copy link

GNU testsuite comparison:

Skipping an intermittent issue tests/misc/tee (passes in this run but fails in the 'main' branch)

@gabelluardo
Copy link
Contributor Author

I rebased and aligned it with the current main branch. If it's all good, I think we can merge on main now

@github-actions
Copy link

GNU testsuite comparison:

Skipping an intermittent issue tests/misc/stdbuf (passes in this run but fails in the 'main' branch)

@RenjiSann RenjiSann merged commit a72a1c7 into uutils:main Jul 17, 2025
73 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