Skip to content

Conversation

jesse-shopify
Copy link

@jesse-shopify jesse-shopify commented Aug 8, 2025

Move clippy lint configuration (which is declarative) from lib.rs into Cargo.toml.

Lint Groups

Forbid: correctness, suspicious, perf
Deny: complexity
Allow: pedantic, style, restricted

Can be overridden with allow in code or with specific lint?

  • Deny = Yes
  • Forbid = No

Individual Lints

Allow from complexity

too_many_arguments = "allow"
identity_op = "allow"

Deny from allowed groups

ptr_arg = "forbid"
ref_option = "forbid"
inefficient_to_string = "forbid"
from_over_into = "forbid"
unwrap_or_default = "forbid"
redundant_static_lifetimes = "forbid"
match_like_matches_macro = "forbid"
field_reassign_with_default = "forbid"
inherent_to_string = "forbid"
owned_cow = "forbid"
redundant_clone = "forbid"
str_to_string = "forbid"
single_char_pattern = "forbid"
single_char_add_str = "forbid"
unnecessary_owned_empty_strings = "forbid"
manual_string_new = "forbid"

Resources

@jesse-shopify jesse-shopify changed the title YJIT, ZJIT: Update Clippy settings and fix lint YJIT, ZJIT: enhance clippy settings and fix lint Aug 8, 2025
@jesse-shopify jesse-shopify force-pushed the clippy branch 16 times, most recently from 1bb890c to 0290ccb Compare August 11, 2025 19:40
use them in yjit and zjit
fix clippy issues
@jesse-shopify jesse-shopify marked this pull request as ready for review August 11, 2025 23:00
@jesse-shopify jesse-shopify marked this pull request as draft August 12, 2025 07:23
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.

1 participant