+
Skip to content

Enable some pedantic rules #8203

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 3 commits into from
Jul 17, 2025
Merged

Enable some pedantic rules #8203

merged 3 commits into from
Jul 17, 2025

Conversation

gabelluardo
Copy link
Contributor

Ref #4949 #8201

@github-actions 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)

sylvestre
sylvestre reviewed Jun 17, 2025
View reviewed changes
sylvestre
sylvestre reviewed Jun 17, 2025
View reviewed changes
@github-actions GitHub Actions
Copy link

GNU testsuite comparison:

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

RenjiSann
RenjiSann reviewed Jun 20, 2025
View reviewed changes
@github-actions 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 reviewed Jul 15, 2025
View reviewed changes
RenjiSann
RenjiSann reviewed Jul 16, 2025
View reviewed changes
@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 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 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
Reviewers

@sylvestre sylvestre sylvestre left review comments

@RenjiSann RenjiSann RenjiSann left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@gabelluardo @RenjiSann @sylvestre
点击 这是indexloc提供的php浏览器服务，不要输入任何密码和下载