Skip to content

Conversation

@matklad
Copy link
Contributor

@matklad matklad commented Feb 10, 2018

bors r+

bors bot added a commit that referenced this pull request Feb 10, 2018
46: Names r=matklad a=matklad

bors r+
@matklad
Copy link
Contributor Author

matklad commented Feb 10, 2018

bors r-

@bors
Copy link
Contributor

bors bot commented Feb 10, 2018

Canceled

@matklad
Copy link
Contributor Author

matklad commented Feb 10, 2018

bors r+

bors bot added a commit that referenced this pull request Feb 10, 2018
46: Names r=matklad a=matklad

bors r+
@bors
Copy link
Contributor

bors bot commented Feb 10, 2018

@bors bors bot merged commit 199b3a1 into master Feb 10, 2018
@bors bors bot deleted the names branch February 10, 2018 11:27
Veykril added a commit to Veykril/rust-analyzer that referenced this pull request Nov 7, 2025
46: Use new optional dependency feature syntax making serde truly optional r=Veykril a=Veykril

Fixes rust-analyzer/smol_str#42
bors r+

Co-authored-by: Lukas Wirth <[email protected]>
Veykril added a commit to Veykril/rust-analyzer that referenced this pull request Nov 7, 2025
46: Fix some rust.ungram inconsistencies r=Veykril a=Veykril



Co-authored-by: Lukas Wirth <[email protected]>
Veykril pushed a commit to Veykril/rust-analyzer that referenced this pull request Nov 7, 2025
Veykril pushed a commit to Veykril/rust-analyzer that referenced this pull request Nov 7, 2025
48: Revert the visibility removal of macro_rules visibility (rust-lang#46) r=Veykril a=ChayimFriedman2

As discussed in rust-analyzer/ungrammar#46 (comment).

Co-authored-by: Chayim Refael Friedman <[email protected]>
Veykril pushed a commit to Veykril/rust-analyzer that referenced this pull request Nov 7, 2025
46: Fix TextSize::of docs to no longer suggest implementing TextLen r=matklad a=CAD97

Addresses the misleading docs part of rust-lang#45 by editing the docs back to their minimum. Does not yet provide advice on idioms for non-primitive types.

Co-authored-by: CAD97 <[email protected]>
lnicola pushed a commit to lnicola/rust-analyzer that referenced this pull request Nov 11, 2025
lnicola pushed a commit to lnicola/rust-analyzer that referenced this pull request Nov 15, 2025
lnicola pushed a commit to lnicola/rust-analyzer that referenced this pull request Nov 15, 2025
48: Revert the visibility removal of macro_rules visibility (rust-lang#46) r=Veykril a=ChayimFriedman2

As discussed in rust-analyzer/ungrammar#46 (comment).

Co-authored-by: Chayim Refael Friedman <[email protected]>
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.

2 participants