-
Notifications
You must be signed in to change notification settings - Fork 12.8k
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
test(patterns): add patterns feature tests to borrowck test suite #69817
test(patterns): add patterns feature tests to borrowck test suite #69817
Conversation
@Centril it looks like the slice_patterns and or_patterns were already covered in the borrowck test suite |
@thekuom Ah, sorry; it seems I was unclear. There are already a bunch of borrowck tests in |
@Centril, no worries! I will do those |
This comment has been minimized.
This comment has been minimized.
Thanks; can you squash the commits into one? r=me with that done. |
Adds borrowck tests for the following features: - bindings_after_at - or_patterns - slice_patterns - box_patterns
2ade48f
to
b974d6f
Compare
Done, r=@Centril |
@bors r+ |
📌 Commit b974d6f has been approved by |
🌲 The tree is currently closed for pull requests below priority 1000, this pull request will be tested once the tree is reopened |
Rollup of 10 pull requests Successful merges: - #69475 (Remove the `no_force` query attribute) - #69514 (Remove spotlight) - #69677 (rustc_metadata: Give decoder access to whole crate store) - #69714 (Make PlaceRef take just one lifetime) - #69799 (Allow ZSTs in `AllocRef`) - #69817 (test(patterns): add patterns feature tests to borrowck test suite) - #69836 (Check if output is immediate value) - #69847 (clean up E0393 explanation) - #69861 (Add note about localization to std::fmt docs) - #69877 (Vec::new is const stable in 1.39 not 1.32) Failed merges: r? @ghost
Addresses request here: #69690 (comment)
Fixes #67311.
r? @Centril