Skip to content

fix: make HintRef lifetimes explicit - #5653

Open
jay23606 wants to merge 1 commit into
pyrefly-org:mainfrom
jay23606:fix/5590-explicit-hintref-lifetimes
Open

jay23606 wants to merge 1 commit into
pyrefly-org:mainfrom
jay23606:fix/5590-explicit-hintref-lifetimes

Conversation

@jay23606

@jay23606 jay23606 commented Nov 14, 2026 •

Copy link
Copy Markdown

Summary

Makes the lifetime parameters on HintRef explicit in as_hint_for_display, removing the inconsistent lifetime syntax warning from issue #5590.

Fixes #5590

  • I am a human, and I fully understand the code I am submitting.

Test Plan

  • rustfmt --check pyrefly/lib/alt/operators.rs — passed.
  • cargo check -p pyrefly — not completed locally; dependency fetching stalled. Draft CI is needed to confirm the compiler warning is gone.

Use explicit elided lifetime placeholders on the HintRef return type so the lifetime relationships are clear and the compiler warning is removed. Fixes pyrefly-org#5590.
@jay23606
jay23606 marked this pull request as ready for review November 14, 2026 18:55

This branch has not been deployed

No deployments
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.

warning: hiding a lifetime that's elided elsewhere is confusing

1 participant