Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Merge #6834 | bors[bot] | 2020-12-11 | 1 | -3/+3 |
|\ | | | | | | | | | | | | | | | | | | | | | | | | | 6834: Use Attrs::docs in runnables instead of DocCommentsOwner r=kjeremy a=Veykril I figured that we should probably move as much of the doc usage to the HIR as possible hence this PR. If we should keep this AST-based feel free to close. This change does have the nice(but not really useful as I doubt anyones gonna write doc tests like these) side effect that these two doc string snippets allow being run now. ![image](https://user-images.githubusercontent.com/3757771/101945607-bf241400-3bee-11eb-96ce-ccae80028b1f.png) ![image](https://user-images.githubusercontent.com/3757771/101946375-2e9a0380-3bef-11eb-9950-e35168fdd048.png) Co-authored-by: Lukas Wirth <[email protected]> | ||||
| * | Use Attrs::docs in runnables instead of DocCommentsOwner | Lukas Wirth | 2020-12-11 | 1 | -3/+3 |
| | | |||||
* | | Add upstream commit to builtin_attr.rs | Jonas Schievink | 2020-12-11 | 1 | -0/+2 |
| | | |||||
* | | Add builtin attributes for use in nameres | Jonas Schievink | 2020-12-11 | 2 | -0/+429 |
|/ | |||||
* | Remove item tree tests | Jonas Schievink | 2020-12-10 | 2 | -441/+0 |
| | | | | | They were useful during initial development of the item tree, but now just cause churn | ||||
* | Ignore extern items in incorrect-case check | Jonas Schievink | 2020-12-10 | 3 | -5/+17 |
| | |||||
* | Introduce anchored_path | Aleksey Kladov | 2020-12-09 | 2 | -5/+6 |
| | | | | | They allow to represent paths like `#[path = "C:\path.rs"] mod foo;` in a lossless cross-platform & network-transparent way. | ||||
* | Merge #6771 | bors[bot] | 2020-12-09 | 1 | -5/+47 |
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6771: Properly attach inner attributes in Attrs::new r=matklad a=Veykril Properly attach inner and outer attributes to the things they actually belong to in the HIR. ~~I can add some tests for this if wanted once I know where to put them/how to test for this.~~ Put some tests into `hover.rs`. So the following snippet ```rust mod foo { //! Hello } ``` now shows `Hello` on hover 🎉 Fixes #2148 Co-authored-by: Lukas Wirth <[email protected]> | ||||
| * | Properly fetch inner and outer docs on hir-level | Lukas Wirth | 2020-12-09 | 1 | -17/+25 |
| | | |||||
| * | Properly fetch inner and outer attributes on hir-level | Lukas Wirth | 2020-12-08 | 1 | -5/+39 |
| | | |||||
* | | Merge #6774 | bors[bot] | 2020-12-09 | 1 | -2/+2 |
|\ \ | | | | | | | | | | | | | | | | | | | | | | 6774: Increment recursion count in Expander only on success r=lnicola a=Veykril Fixes #6764 Co-authored-by: Lukas Wirth <[email protected]> | ||||
| * | | Properly decrement recursion count in Expander | Lukas Wirth | 2020-12-09 | 1 | -2/+2 |
| |/ | |||||
* | | Use `original_file_range` in `TestDB` | Jonas Schievink | 2020-12-08 | 1 | -9/+3 |
| | | |||||
* | | Add test for `$crate` in builtin macros | Jonas Schievink | 2020-12-08 | 1 | -0/+28 |
| | | | | | | | | Fixes #6716 | ||||
* | | Handle macros in `TestDB::check_diagnostics` | Jonas Schievink | 2020-12-08 | 1 | -2/+9 |
|/ | |||||
* | Escape string literals in Attr::from_src | Lukas Wirth | 2020-12-08 | 2 | -12/+6 |
| | |||||
* | Replace Arc<[str]> with String in attr::Documentation | Lukas Wirth | 2020-12-07 | 1 | -2/+2 |
| | |||||
* | Keep doc attribute order | Lukas Wirth | 2020-12-07 | 1 | -14/+26 |
| | |||||
* | Remove raw pre and suffixes from string attr literals | Lukas Wirth | 2020-12-07 | 1 | -2/+5 |
| | |||||
* | Remove hir_def/docs.rs module | Lukas Wirth | 2020-12-07 | 3 | -83/+22 |
| | |||||
* | Remove documentation query | Lukas Wirth | 2020-12-07 | 3 | -51/+19 |
| | |||||
* | Don't insert blank lines between doc attributes | Lukas Wirth | 2020-12-07 | 1 | -11/+11 |
| | |||||
* | Make `compile_error!` message match upstream rustc | Jonas Schievink | 2020-12-03 | 1 | -1/+1 |
| | | | | It only consists of the argument passed to it | ||||
* | Fix proc macro token mapping | Jonas Schievink | 2020-12-03 | 1 | -2/+2 |
| | |||||
* | Make `compile_error!` lazy and emit a diagnostic | Jonas Schievink | 2020-12-03 | 1 | -0/+6 |
| | |||||
* | Give better diagnostic if `OUT_DIR` is unset | Jonas Schievink | 2020-12-03 | 1 | -0/+3 |
| | |||||
* | Update/Fix tests | Jonas Schievink | 2020-12-03 | 1 | -4/+15 |
| | |||||
* | Propagate eager expansion errors | Jonas Schievink | 2020-12-03 | 2 | -17/+54 |
| | |||||
* | Test macro diagnostics in body lowering | Jonas Schievink | 2020-12-02 | 1 | -0/+41 |
| | |||||
* | Merge #6698 | bors[bot] | 2020-12-02 | 1 | -2/+5 |
|\ | | | | | | | | | | | | | | | | | | | | | | | 6698: Attach macro expansion errors to the right file r=jonas-schievink a=jonas-schievink Previously it attached them to the result of the macro expansion (or, if no result was produced, to the file containing the invocation). Always use the file containing the invocation. This doesn't seem to have any observable difference, but seems better in theory. bors r+ Co-authored-by: Jonas Schievink <[email protected]> | ||||
| * | Attach macro expansion errors to the right file | Jonas Schievink | 2020-12-02 | 1 | -2/+5 |
| | | |||||
* | | Merge #6697 | bors[bot] | 2020-12-02 | 2 | -1/+22 |
|\ \ | |/ |/| | | | | | | | | | | | 6697: Don't discard PathKind::Abs information in lower_use::convert_path r=matklad a=Veykril Fixes #6694 Co-authored-by: Lukas Wirth <[email protected]> | ||||
| * | Don't discard PathKind::Abs information in lower_use::convert_path | Lukas Wirth | 2020-12-02 | 2 | -1/+22 |
| | | |||||
* | | Emit unresolved proc macro errors | Jonas Schievink | 2020-12-01 | 2 | -9/+28 |
| | | |||||
* | | Emit macro diagnostics when lowering bodies | Jonas Schievink | 2020-11-30 | 4 | -27/+68 |
|/ | |||||
* | More accurately place proc-macro diagnostic | Jonas Schievink | 2020-11-27 | 2 | -4/+42 |
| | |||||
* | Publish diagnostics for macro expansion errors | Jonas Schievink | 2020-11-27 | 3 | -4/+137 |
| | |||||
* | Don't store `SyntaxNodePtr` in `CrateDefMap` | Jonas Schievink | 2020-11-26 | 2 | -8/+7 |
| | | | | It is volatile across reparses and makes incrementality worse. | ||||
* | Test def map invalidation with #[cfg] below change | Jonas Schievink | 2020-11-26 | 1 | -0/+6 |
| | |||||
* | Improve autoimports on completion speed | Kirill Bulatov | 2020-11-24 | 1 | -5/+77 |
| | | | | | * Ignore modules eaferly * Do less completion string rendering | ||||
* | Cleanup API | Aleksey Kladov | 2020-11-06 | 1 | -10/+11 |
| | |||||
* | Don't stack overflow on circular modules | Aleksey Kladov | 2020-11-04 | 4 | -18/+62 |
| | | | | closes #6453 | ||||
* | Merge #6435 | bors[bot] | 2020-11-03 | 1 | -2/+15 |
|\ | | | | | | | | | | | | | | | 6435: Test Fixture ExplicitRoot + ModuleResolutionRelativePathOutsideRoot. r=matklad a=rickvanprim Updates `module_resolution_relative_path_outside_root` test to check valid paths outside of the root, by moving the root to a subpath so that paths outside of it are possible. If this would be more appropriate as a new test, or if the original check for an invalid path should be left, I'm happy to update. Co-authored-by: James Leitch <[email protected]> | ||||
| * | Feedback. | James Leitch | 2020-11-02 | 1 | -1/+1 |
| | | |||||
| * | Test Fixture ExplicitRoot + ModuleResolutionRelativePathOutsideRoot. | James Leitch | 2020-11-02 | 1 | -2/+15 |
| | | |||||
* | | Remove more unreachable pubs | Aleksey Kladov | 2020-11-02 | 1 | -2/+2 |
| | | |||||
* | | Deny unreachable-pub | Aleksey Kladov | 2020-11-02 | 4 | -20/+20 |
| | | | | | | | | | | | | | | | | It's very useful when `pub` is equivalent to "this is crate's public API", let's enforce this! Ideally, we should enforce it for local `cargo test`, and only during CI, but that needs https://github.com/rust-lang/cargo/issues/5034. | ||||
* | | Handle #![cfg] in crate root | Jonas Schievink | 2020-10-26 | 2 | -2/+22 |
|/ | |||||
* | Fix name of InactiveCode diagnostic | Jonas Schievink | 2020-10-23 | 1 | -1/+1 |
| | |||||
* | Diagnose #[cfg]s in bodies | Jonas Schievink | 2020-10-23 | 7 | -94/+213 |
| |