Skip to content

Improve placeholder scanning for panic macros#1826

Merged
oferchen merged 1 commit into
masterfrom
establish-rsync-3.4.1-rust-equivalence
Nov 1, 2025
Merged

Improve placeholder scanning for panic macros#1826
oferchen merged 1 commit into
masterfrom
establish-rsync-3.4.1-rust-equivalence

Conversation

@oferchen
Copy link
Copy Markdown
Owner

@oferchen oferchen commented Nov 1, 2025

Summary

  • extend the no-placeholders scan to track panic! invocations across multiple lines and handle delimiter comments
  • ensure placeholder words inside panic! blocks (even after comments) trigger a violation
  • add coverage for multi-line panic! invocations in the xtask test suite

Testing

  • cargo test -p xtask

https://chatgpt.com/codex/tasks/task_e_69055719b8b48323955538d6840254c9

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant