Skip to content

Conversation

magistau
Copy link
Contributor

@magistau magistau commented Jul 17, 2025

First part of #779, contains only the most basic stuff

@magistau magistau force-pushed the clippy-init branch 6 times, most recently from 154a2d2 to c86554b Compare July 18, 2025 03:03
@magistau magistau marked this pull request as ready for review July 18, 2025 03:36
@magistau magistau force-pushed the clippy-init branch 4 times, most recently from c6c1c02 to 7d29ecf Compare July 22, 2025 07:01
@magistau
Copy link
Contributor Author

Updated to fix violations in #783 (see 7589028)

Copy link
Member

Choose a reason for hiding this comment

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

Why changes for this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

As the commit message says, to make Clippy run on all packages rather than the library target of the main package. Otherwise, there's no linting done on src/main.rs and other crates like uiua-parser, which is rather surprising.

@magistau magistau force-pushed the clippy-init branch 3 times, most recently from c1b66a8 to 078735d Compare August 5, 2025 18:20
@magistau magistau marked this pull request as draft August 5, 2025 18:26
@magistau magistau force-pushed the clippy-init branch 7 times, most recently from ca98674 to 862d001 Compare August 5, 2025 19:02
@magistau magistau marked this pull request as ready for review August 5, 2025 19:11
@magistau magistau requested a review from kaikalii August 5, 2025 19:11
@kaikalii kaikalii merged commit dcf279d into uiua-lang:main Aug 7, 2025
5 checks passed
@magistau magistau deleted the clippy-init branch August 7, 2025 02:35
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