-
Notifications
You must be signed in to change notification settings - Fork 13.3k
New Guide: crates and modules #15956
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
Conversation
# Lambdas | ||
Rust features a strong module system, but it works a bit differently than in | ||
other programming languages. Rust's module system has two main components: | ||
**crate**s, and **module**s. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
**crates**, and **module**s
doesn't seem to be rendered at least in github's preview
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've used it elsewhere, so it does work.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Might be that it doesn't like star delimiters in the middle of a word.
Anyways, I'd suggest to embold the whole word. Bold makes the letters wider so having just a part emboldened seems just weird.
I've squashed this up, if we're happy with that last sentence, this is good to go. |
Whoops. Fixed the one test failure. |
Also fixes a couple of filesystem references that I forgot to change. 😅
} | ||
``` | ||
|
||
When we include a module like this, we don't need to make the `mod` declaration, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"the mod declaration" is at best ambiguous here. It could be interpreted as referring to the mod decl in lib.rs, which of course cannot be omitted
Improve completion label details display This PR improves completion label details display by separating trait and alias info from the `label` LSP field to an additional existing `label_detail` field. Changes look like the following: ### Before  ### After  _All existing tests are passed without any changes in test themselves logic._
fix: add fallback for completion label details This PR adds a fallback to a previous implementation in a case when the label detail field isn't supported by LSP client and the support isn't reported by the LSP initialize request. In this case additional info about trait and aliases would be merged into the label field as it was before the rust-lang#15956 PR.
Also fixes a couple of filesystem references that I forgot to change. 😅