Skip to content
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.

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

fix: clarify suggestion that &T must refer to T: Sync for &T: Send #87322

Merged
merged 2 commits into from
Jul 23, 2021

Commits on Jul 22, 2021

  1. Squash all commits.

    add test for issue 86507
    
    add stderr for issue 86507
    
    update issue-86507 UI test
    
    add comment for the expected error in UI test file
    
    add proper 'refers to <ref_type>' in suggestion
    
    update diagnostic phrasing; update test to match new phrasing; re-organize logic for checking T: Sync
    
    evaluate additional obligation to figure out if T is Sync
    
    run './x.py test tidy --bless'
    
    incorporate changes from review; reorganize logic for readability
    chazkiker2 committed Jul 22, 2021
    Configuration menu
    Copy the full SHA
    831ac19 View commit details
    Browse the repository at this point in the history

Commits on Jul 23, 2021

  1. update clippy ui test 'future_not_send.stderr' to match

    the new diagnostic messages
    chazkiker2 committed Jul 23, 2021
    Configuration menu
    Copy the full SHA
    a1518f0 View commit details
    Browse the repository at this point in the history