-
Notifications
You must be signed in to change notification settings - Fork 13.3k
Re-add rust-analyzer as a subtree #99465
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
Closed
Closed
Changes from 250 commits
Commits
Show all changes
10000 commits
Select commit
Hold shift + click to select a range
6e9c963
internal: Lift out IdentContext from CompletionContext
Veykril a1ff3ca
Auto merge of #12594 - Veykril:completions, r=Veykril
bors 1f02840
fix: Don't trigger pattern completions when typing a wildcard pattern
Veykril 9fdfa9f
Auto merge of #12596 - Veykril:completions, r=Veykril
bors 06ee4d6
fix: Fix auto-ref completions inserting into wrong locations
Veykril 98a5811
Only apply `cfg(test)` for local crates
flodiebold 46d2271
Remove pattern rendering hack
Veykril 05ca0a2
Refactor a bit
flodiebold 8b07898
Reimplement auto-ref completions for fields
Veykril 439a513
Auto merge of #12597 - Veykril:completions, r=Veykril
bors 07d78b6
Fix test
flodiebold 312ac83
Auto merge of #12599 - flodiebold:no-test-deps, r=flodiebold
bors 7a0774d
internal: Simplify some completions
Veykril 6fc5c3c
Auto merge of #12604 - Veykril:completions, r=Veykril
bors c6f776c
determine doc link type from start instead of text or code
erhuve b7e3f52
handle autolink as edge case
erhuve f780145
apply suggestions
soruh f9379df
add use_trivial_contructor.rs
soruh 817082c
add doc strings to use_trivial_contructor.rs
soruh 96ff235
condense matches on autolink
erhuve 1ef5e14
goto where trait method impl
bitgaoshu 1064c75
reformat code
bitgaoshu 29f01cd
Various cleanups
flodiebold f410fdf
Add two more tests
flodiebold 36fadc4
Fix unstable feature use
flodiebold 472ae16
fix: completes non exhaustive variant within the defining crate
yue4u 19fad54
Auto merge of #12625 - yue4u:fix/non-exhaustive-variant, r=Veykril
bors e0c17e8
fix: trailing ':' on empty inactive reasons
CuriousCorrelation 2642f64
internal: Simplify
Veykril 32b40de
Auto merge of #12628 - Veykril:simplify, r=Veykril
bors 21642a2
Try to publish releases to OpenVSX
lnicola 5107123
clarify comment and add autolink test case
erhuve 434e718
Auto merge of #12605 - erhuve:fix/determine-doc-link-type-at-start, r…
bors 6ecabe3
functions resolve to impl
bitgaoshu 9e6bff7
fix some test due to resolve to where trait m impl
bitgaoshu c80c348
Improve proc macro errors a bit
flodiebold 45fd5e6
Improve comments
flodiebold 401a71d
Auto merge of #12629 - flodiebold:proc-macro-error-improvement, r=Vey…
bors 9ea8d58
fix test in qualify_method: stay in trait path
bitgaoshu 353829f
highlight: trait path
bitgaoshu 900b420
rename "trait_def" to "def_in_trait"
bitgaoshu e3b1303
fix test in resolve_proc_macro: Foo is Owner
bitgaoshu 5bb123d
Auto merge of #12520 - Veykril:flycheck-cancel, r=Veykril
bors 82c1b31
restore sth
bitgaoshu 0dbc091
add test for suggest_name
bitgaoshu fb6278e
Reduce intermediate allocations while printing witnesses
iDawer e4e2a46
Correct target_feature completion
antglb f536766
complete raw identifier with "r#" prefix
xuhongxu96 45b8960
Fix typo in build.rs
b74e96f
Auto merge of #12638 - buffet:fix-typo, r=flodiebold
bors 408fa14
add test for item same name
bitgaoshu dcb4837
WellFormed -> Holds
bitgaoshu bef89e9
Udate unit tests
antglb 7f1fac7
fix: deduplicate cfg completions
weirdsmiley f6c0bf8
Auto merge of #12642 - weirdsmiley:master, r=Veykril
bors 4c39546
Auto merge of #12635 - antogilbert:tgt_feat, r=Veykril
bors bdbffdd
fix: Fix completions for locals not working properly inside macro calls
Veykril 9eaf96c
Auto merge of #12643 - Veykril:macro-compl, r=Veykril
bors 8b3ec12
fix: Report proc macro errors in expressions correctly as well
flodiebold 6407727
Auto merge of #12648 - flodiebold:proc-macro-errors-again, r=flodiebold
bors 36d2b43
fix: improve whitespace insertion in pretty printer
lowr b0102bd
Auto merge of #12652 - lnicola:openvsx, r=lnicola
bors 979d951
Ignore the `bin` artifact for `bench` targets
ChayimFriedman2 2ff505a
Auto merge of #12428 - lowr:experimental/destructuring-assignment, r=…
bors 7f9c054
Auto merge of #12626 - CuriousCorrelation:fix/empty-reasons, r=flodie…
bors 461c0cc
Correct wording
iDawer e417992
Add static assertions for some unreachble paths
iDawer 6420840
Auto merge of #12634 - iDawer:match-check.witnesses, r=flodiebold
bors ce36446
Auto merge of #12655 - ChayimFriedman2:debug-bench-filter-bin-target,…
bors ddddca8
fix: Fix flycheck sending cancel progress with no running process
Veykril 7d0e5ae
Auto merge of #12660 - Veykril:flycheck, r=Veykril
bors f8c416e
Auto merge of #12650 - lowr:fix/12591, r=lnicola
bors 531e152
fix: Simplify macro statement expansion handling
Veykril 9165e3b
Update hir-ty test outputs
Veykril 58d5c69
Fix Expr::MacroStmts using wrong scopes
Veykril 8e764a8
fix: Fix attribute macros on assoc items being discarded with disable…
Veykril e5e5a09
Fix blocks not considering stmt without semi as tails
Veykril 8489cd7
Auto merge of #12670 - Veykril:assoc-attrs, r=Veykril
bors ed44fe5
Auto merge of #12668 - Veykril:mac-source-map, r=Veykril
bors 994f3cf
Auto merge of #12636 - xuhongxu96:fix-12148, r=Veykril
bors e715195
Add tests for #12669
flodiebold 9a12d0d
Fix case of ignored/broken proc macro
flodiebold ec78b6b
Auto merge of #12671 - flodiebold:test-for-12669, r=flodiebold
bors 622b516
fix: variants rendering in pattern path
yue4u ea7ea70
fix: escape for enum variant
yue4u cd42b20
fix: Trigger flyimport completions in item lists again
Veykril d4b79ad
Auto merge of #12678 - Veykril:flyimport, r=Veykril
bors f2963cf
fix: typos in hir-ty
a-kenji 30680df
Auto merge of #12679 - a-kenji:fix-typos-hir, r=lnicola
bors d101439
Auto merge of #12627 - yue4u:fix/struct-variant-patterns, r=Veykril
bors afdbd6c
fix: infer lhs first on ordinary assignment expressions
lowr 649e1f5
fix: report type mismatch on identifier in destructuring assignments
lowr d1ac462
Auto merge of #12680 - lowr:fix/12428-regression, r=Veykril
bors 0039d6f
fix: Extract Function produces duplicate fn names
DorianListens cc0bb71
Auto merge of #12662 - DorianListens:dscheidt/extract-function-duplic…
bors e394000
fix: Extract function from trait impl
DorianListens 212f84a
Bump crossbeam-channel
lnicola 416787d
Bump anyhow
lnicola c7c314d
Bump quote
lnicola e6fcb23
Bump either
lnicola 2af61bc
Bump pulldown-cmark-to-cmark
lnicola 791f2a0
Bump smallvec
lnicola 87572be
Bump serde
lnicola b24ece5
Bump serde_json
lnicola 6669f38
Bump indexmap
lnicola 2d3f0b4
Bump semver
lnicola 9e7ca80
Bump cargo_metadata
lnicola a3011e5
Bump arbitrary and derive_arbitrary
lnicola 5381811
Bump tracing-subscriber
lnicola e05e6c6
Bump object
lnicola 78beb4c
Bump chalk
lnicola 993a19a
Bump the rest of the deps
lnicola 75b2232
Auto merge of #12681 - lnicola:bump-deps, r=lnicola
bors ccf854b
Improve documentation for buildScripts.overrideCommand / checkOnSave.…
flodiebold 5235732
Add back restartServerOnConfigChange option
flodiebold e1a8c0b
Auto merge of #12687 - flodiebold:override-docs, r=flodiebold
bors db49ac8
internal: Record all macro definitions in ItemScope
Veykril 383ee6a
internal: Update inert attribute list
Veykril 6edf624
Auto merge of #12690 - Veykril:inert-attrs, r=Veykril
bors 976d07e
fix: Fix unresolved proc macro diagnostics pointing to macro expansions
Veykril 2926ca0
Remove useless bors.toml
0xPoe 0d9737a
Fix project root assert
0xPoe 441e659
Complete associated type only in trait generic arg
xuhongxu96 fee5555
Auto merge of #12694 - hi-rustin:rustin-patch-bors, r=jonas-schievink
bors b9ba9fa
Add str_ref_to_string fix
0xPoe 0f2eba5
Show only assoc type args in the correct arg pos
xuhongxu96 ad5a851
Code: use different schemes for the custom views
lnicola 00194ad
Auto merge of #12702 - lnicola:vscode-schemes, r=lnicola
bors 75fb3de
Handle generic args per arg index
xuhongxu96 0360ed5
check arg_idx >= n_params only if arg_idx >= n_required_params
xuhongxu96 d2fd137
Use `SmallVec` to slightly shrink `ModPath` size
c46570e
Auto merge of #12704 - jonas-schievink:smol-paths, r=jonas-schievink
bors 3248601
fix default type param value position
xuhongxu96 c296e77
Auto merge of #12695 - xuhongxu96:fix-12140, r=jonas-schievink
bors f0af913
publish workflow: Fix names for existing crates
davidlattimore 6f819e3
Remove unnecessary unsafe from format_args expansion
davidlattimore 09abd76
Auto merge of #12716 - davidlattimore:fix-publish-crate-names, r=lnicola
bors 6c6ae96
Update remaining GitHub URLs
df4ed94
Auto merge of #12722 - jonas-schievink:update-urls, r=jonas-schievink
bors 7181a39
Auto merge of #12676 - DorianListens:dscheidt/extract-fun-trait-impl,…
bors c419aa9
Auto merge of #12719 - davidlattimore:format-args-no-unsafe, r=jonas-…
bors 603b6fc
fix: Extract Function misses locals used in closures
DorianListens 2836dd1
Auto merge of #12706 - DorianListens:dscheidt/closure-args, r=DorianL…
bors 21062f9
fix: Improve suggested names for extracted variables
DorianListens 638abba
Fix test
0xPoe 666343b
Auto merge of #12727 - DorianListens:dscheidt/extract-var-field-name,…
bors 03a62c1
Bump vscode-languageclient
lnicola b43708c
Bump d3 and d3-graphviz
lnicola 1b138b1
Bump esbuild
lnicola 7b1b711
Bump eslint
lnicola 0e38961
Bump vsce
lnicola d03ed83
Bump @typescript-eslint
lnicola 8e889ac
Bump prettier
lnicola 2c9e395
Bump typescript and tslib
lnicola 358d6c6
Bump @types/node to 16
lnicola a059e79
Bump @vscode/test-electron
lnicola cb38145
Bump transitive npm deps
lnicola 5342f47
Auto merge of #12732 - lnicola:bump-npm, r=lnicola
bors 4b3164f
fix: ignore renames for crate root
harpsword eaebead
Fix config keys regarding imports in docs
TonalidadeHidrica 6e55716
Fix obsolete `loadOutDirsFromCheck` configs
TonalidadeHidrica 195ceea
Fix obsolete `diagnostics.experimental.enable`
TonalidadeHidrica f0649f2
Fix `rustcSource` -> `rustc_source`
TonalidadeHidrica dfcbed0
Fix obsolete `enableRangeFormatting` config
TonalidadeHidrica e3bde57
Fix obsolete `completion.snippets` config
TonalidadeHidrica d9ab7f2
Fix typos
0xPoe f7bb932
Auto merge of #12744 - hi-rustin:rustin-patch-typos, r=lnicola
bors df66eb7
Implement `ignore` and `index` metavar expression
caf23f2
Auto merge of #12745 - jonas-schievink:metavar-exprs, r=jonas-schievink
bors ebfbb31
Update 1.63 proc macro ABI to match rustc
4cbf23c
Auto merge of #12747 - jonas-schievink:proc-macro-abi-1.63, r=jonas-s…
bors 84a6bc9
Auto merge of #12742 - TonalidadeHidrica:fix-obsolete-config-keys, r=…
bors fc47ce5
Auto merge of #12696 - hi-rustin:rustin-patch-fix, r=Veykril
bors def89af
fix: Fix VSCode status bar tooltip not showing the error messages
Veykril 90b5097
Auto merge of #12754 - Veykril:vscode-status, r=Veykril
bors ec51dcb
fix: Set server binary version when distributing
Veykril 794ecd5
Auto merge of #12755 - Veykril:server-version, r=lnicola
bors a5ad4de
add tests
soruh 075ab03
fix: Support generics in extract_function assist
DorianListens 796641b
Make search for applicable generics more precise
DorianListens 4cbde4e
fix(completion): `super::` completion at crate root and module depth
feniljain 5af3ef5
Auto merge of #12735 - feniljain:fix-completions, r=feniljain
bors fbba1d7
Auto merge of #12691 - Veykril:proc-macro-diag, r=Veykril
bors 073b325
Auto merge of #12556 - DorianListens:dscheidt/generic-extract, r=Veykril
bors 6b823b0
fix: Fix imports being inserted before doc comments in inline modules
Veykril 029184d
Auto merge of #12765 - Veykril:import-insert-fix, r=Veykril
bors 7ff6c36
fix: Don't show qualified path completions for private items
Veykril f4fe2ea
Remove deprecate action
matklad 248fa55
Update .github/workflows/release.yaml
matklad b49f2a2
Remove deprecate action
matklad 4ad9185
Auto merge of #12764 - matklad:rustupaction, r=lnicola
bors 25090f0
internal: Remove allocation in DefCollector::reseed_with_unresolved_a…
Veykril 2e9d5b5
Auto merge of #12766 - Veykril:completion-vis, r=Veykril
bors d3796ad
Auto merge of #12772 - Veykril:nameres, r=Veykril
bors b96f8f1
fix: Improve self param completion applicability
Veykril 96481b7
Auto merge of #12773 - Veykril:self-compl, r=Veykril
bors c205c62
try fixing spurious CI failures on windows
matklad f2c2c36
Auto merge of #12776 - matklad:nosu, r=matklad
bors 0a4065d
Improve syntax fixup a bit, handle incomplete `if`
flodiebold 3d7da51
Auto merge of #12775 - flodiebold:syntax-fixup-if, r=flodiebold
bors ac526e0
Auto merge of #12712 - harpsword:fix-rename-crate-root, r=Veykril
bors 01d2517
Auto merge of #12539 - soruh:instanciate_empty_structs, r=Veykril
bors 766c5f0
Auto merge of #12689 - Veykril:macro-rec, r=Veykril
bors b9b42e8
tests: add hover tests for const generics
Logarithmus a96f0aa
feat: support negative const generic parameters
Logarithmus a0fd58b
Check for local IDs belong to same definition
iDawer 667fd25
Auto merge of #12781 - iDawer:hir_display.stack_overflow, r=lnicola
bors 15f7300
refactor: inline some variables
Logarithmus 83177a7
fix: address suggestions
Logarithmus db6a85d
Auto merge of #12778 - Logarithmus:feature/fix-negative-const-generic…
bors d9336a4
fix: un-inline `ConstScalarOrPath::from_expr_opt`
Logarithmus 897a7ec
Auto merge of #12785 - Logarithmus:feature/fix-negative-const-generic…
bors ea19e70
Fix extract variable assist for subexpression in mutable borrow
hasali19 b5aa3b3
fix: “Generate constant” ignores the path prefix of the identifier
harpsword 8e379ce
Auto merge of #12286 - harpsword:fix_generate_constant, r=Veykril
bors 6cb0746
Support the 1.64 nightly proc macro ABI
ee2d5fe
Auto merge of #12795 - jonas-schievink:proc-macro-abi-1.64, r=jonas-s…
bors be30c4d
chore: change str_ref_to_string to str_ref_to_owned
mmirate e2eaa99
Auto merge of #12788 - hasali19:extract-var-mut, r=jonas-schievink
bors 5ad981b
Remove outdated proc macro ABIs
e01de19
Auto merge of #12798 - jonas-schievink:trim-proc-macro-abis, r=jonas-…
bors 188c0e4
Remove macro ABI version from doc comment
ea41617
Auto merge of #12802 - rust-lang:jonas-schievink-patch-1, r=jonas-sch…
bors ec1142c
Improve file watcher config
38c11be
Move `convert_to_def_in_trait` into `ide-db`
Veykril 22e53f1
Auto merge of #12549 - bitgaoshu:goto_where_trait_m_impl, r=Veykril
bors fac6a64
Auto merge of #12804 - jonas-schievink:config-watcher, r=Veykril
bors 530eb27
Auto merge of #11819 - rust-lang:dependabot/npm_and_yarn/editors/code…
bors d17a5ef
Add simple support for completion item details
Veykril dc05192
Auto merge of #12807 - Veykril:completion-item-details, r=Veykril
bors 567a5e9
Auto merge of #12796 - mmirate:patch-1, r=Veykril
bors 474f5ea
Work around Code bug with empty diagnostics
lnicola 88515b9
Auto merge of #12809 - lnicola:empty-diagnostics, r=lnicola
bors 1b41647
Upgrade to [email protected]
fasterthanlime 307be7c
Add CARGO_WORKSPACE_DIR env var in `.cargo/config.toml`
fasterthanlime 7d20ff3
Auto merge of #12817 - fasterthanlime:expect-test-1-4-0, r=lnicola
bors 0424e75
Remove rust-analyzer git submodule
fasterthanlime 7a03bf4
Add 'src/tools/rust-analyzer/' from commit '7d20ff3ffbd30a89b7baa7cd9…
fasterthanlime 0b33563
Add `./x.py test rust-analyzer`
fasterthanlime 81597bd
Pass --locked when building proc-macro-test-impl
fasterthanlime b189b49
Add (and use) sysroot ABI when building rust-analyzer in-tree
fasterthanlime ba4b8b2
Allow required unstable features for RA sysroot ABI
fasterthanlime 8990e37
Implement check::RustAnalyzer
fasterthanlime c213c23
Apply suggestions from @jyn514 re check tool
fasterthanlime 6150a0a
Revert "Apply suggestions from @jyn514 re check tool"
fasterthanlime 380eeaa
Start syncing with proc macro bridge changes
fasterthanlime 378fc76
Handle bridge=>tt conversions
fasterthanlime a005294
Stub most of the new proc macro interface
fasterthanlime a6214e8
cargo fmt
fasterthanlime 2582a1d
Add failing test for cloning literals
fasterthanlime File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Submodule rust-analyzer
deleted from
897a7e
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,11 @@ | ||
[alias] | ||
xtask = "run --package xtask --bin xtask --" | ||
tq = "test -- -q" | ||
qt = "tq" | ||
lint = "clippy --all-targets -- -Aclippy::collapsible_if -Aclippy::needless_pass_by_value -Aclippy::nonminimal_bool -Aclippy::redundant_pattern_matching --cap-lints warn" | ||
|
||
[target.x86_64-pc-windows-msvc] | ||
linker = "rust-lld" | ||
|
||
[env] | ||
CARGO_WORKSPACE_DIR = { value = "", relative = true } |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,19 @@ | ||
# https://EditorConfig.org | ||
root = true | ||
|
||
[*] | ||
charset = utf-8 | ||
trim_trailing_whitespace = true | ||
end_of_line = lf | ||
insert_final_newline = true | ||
indent_style = space | ||
|
||
[*.{rs,toml}] | ||
indent_size = 4 | ||
|
||
[*.ts] | ||
indent_size = 4 | ||
[*.js] | ||
indent_size = 4 | ||
[*.json] | ||
indent_size = 4 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,8 @@ | ||
# for this file to take effect make sure you use git ^2.23 and | ||
# add ignoreFile to your git configuration: | ||
# ``` | ||
# git config --global blame.ignoreRevsFile .git-blame-ignore-revs | ||
# ``` | ||
|
||
# prettier format | ||
f247090558c9ba3c551566eae5882b7ca865225f |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,8 @@ | ||
* text=auto eol=lf | ||
# git grep shouldn't match entries in this benchmark data | ||
bench_data/** binary | ||
crates/syntax/test_data/** -text eof=LF | ||
# Older git versions try to fix line endings on images, this prevents it. | ||
*.png binary | ||
*.jpg binary | ||
*.ico binary |
10 changes: 10 additions & 0 deletions
10
src/tools/rust-analyzer/.github/ISSUE_TEMPLATE/blank_issue.md
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,10 @@ | ||
--- | ||
name: Blank Issue | ||
about: Create a blank issue. | ||
title: '' | ||
labels: '' | ||
assignees: '' | ||
|
||
--- | ||
|
||
|
26 changes: 26 additions & 0 deletions
26
src/tools/rust-analyzer/.github/ISSUE_TEMPLATE/bug_report.md
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,26 @@ | ||
--- | ||
name: Bug report | ||
about: Create a bug report for rust-analyzer. | ||
title: '' | ||
labels: '' | ||
assignees: '' | ||
|
||
--- | ||
|
||
<!-- | ||
Troubleshooting guide: https://rust-analyzer.github.io/manual.html#troubleshooting | ||
Forum for questions: https://users.rust-lang.org/c/ide/14 | ||
|
||
Before submitting, please make sure that you're not running into one of these known issues: | ||
|
||
1. extension doesn't load in VSCodium: #11080 | ||
2. on-the-fly diagnostics are mostly unimplemented (`cargo check` diagnostics will be shown when saving a file): #3107 | ||
|
||
Otherwise please try to provide information which will help us to fix the issue faster. Minimal reproducible examples with few dependencies are especially lovely <3. | ||
--> | ||
|
||
**rust-analyzer version**: (eg. output of "Rust Analyzer: Show RA Version" command) | ||
|
||
**rustc version**: (eg. output of `rustc -V`) | ||
|
||
**relevant settings**: (eg. client settings, or environment variables like `CARGO`, `RUSTUP_HOME` or `CARGO_HOME`) |
17 changes: 17 additions & 0 deletions
17
src/tools/rust-analyzer/.github/ISSUE_TEMPLATE/critical_nightly_regression.md
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,17 @@ | ||
--- | ||
name: Critical Nightly Regression | ||
about: You are using nightly rust-analyzer and the latest version is unusable. | ||
title: '' | ||
labels: '' | ||
assignees: 'matklad' | ||
|
||
--- | ||
|
||
<!-- | ||
Troubleshooting guide: https://rust-analyzer.github.io/manual.html#troubleshooting | ||
|
||
Please try to provide information which will help us to fix the issue faster. Minimal reproducible examples with few dependencies are especially lovely <3. | ||
--> | ||
|
||
This is a serious regression in nightly and it's important to fix it before the next release. | ||
@matklad, please take a look. |
8 changes: 8 additions & 0 deletions
8
src/tools/rust-analyzer/.github/actions/github-release/Dockerfile
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,8 @@ | ||
FROM node:slim | ||
|
||
COPY . /action | ||
WORKDIR /action | ||
|
||
RUN npm install --production | ||
|
||
ENTRYPOINT ["node", "/action/main.js"] |
21 changes: 21 additions & 0 deletions
21
src/tools/rust-analyzer/.github/actions/github-release/README.md
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,21 @@ | ||
# github-release | ||
|
||
Copy-pasted from | ||
https://github.com/bytecodealliance/wasmtime/tree/8acfdbdd8aa550d1b84e0ce1e6222a6605d14e38/.github/actions/github-release | ||
|
||
An action used to publish GitHub releases for `wasmtime`. | ||
|
||
As of the time of this writing there's a few actions floating around which | ||
perform github releases but they all tend to have their set of drawbacks. | ||
Additionally nothing handles deleting releases which we need for our rolling | ||
`dev` release. | ||
|
||
To handle all this this action rolls-its-own implementation using the | ||
actions/toolkit repository and packages published there. These run in a Docker | ||
container and take various inputs to orchestrate the release from the build. | ||
|
||
More comments can be found in `main.js`. | ||
|
||
Testing this is really hard. If you want to try though run `npm install` and | ||
then `node main.js`. You'll have to configure a bunch of env vars though to get | ||
anything reasonably working. |
15 changes: 15 additions & 0 deletions
15
src/tools/rust-analyzer/.github/actions/github-release/action.yml
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,15 @@ | ||
name: 'wasmtime github releases' | ||
description: 'wasmtime github releases' | ||
inputs: | ||
token: | ||
description: '' | ||
required: true | ||
name: | ||
description: '' | ||
required: true | ||
files: | ||
description: '' | ||
required: true | ||
runs: | ||
using: 'docker' | ||
image: 'Dockerfile' |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Rather then adding this at every call site, you could make RustAnalyzer its own step outside of the macro, which calls ensure(ToolBuild) internally with the right features.
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'm now working on the real PR and I'm trying to to understand this comment.
Do you mean adding a separate struct (what would it be named?) that implements
Step
manually? And the macro-generatedStep
would call it, likeRls
ensuresClippy
?I'm trying to wrap my mind around this.
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.
Ohh do you mean not using the macro at all?
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.
correct, not using the macro at all. or you can change the macro not to have
extra_features
in some cases only; but that seems more complicated. up to you which you want to do.