Skip to content

Commit e6c352a

Browse files
authored
Merge pull request #6133 from dibarbet/merge_main
Snap main to release
2 parents 7830baf + ffa697e commit e6c352a

File tree

72 files changed

+4924
-829
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

72 files changed

+4924
-829
lines changed

CHANGELOG.md

Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,25 @@
99
- Debug from .csproj and .sln [#5876](https://github.com/dotnet/vscode-csharp/issues/5876)
1010

1111
## Latest
12+
* Update Roslyn (PR: [#6131](https://github.com/dotnet/vscode-csharp/pull/6131))
13+
* Only show toast for project load failures (PR: [#69494](https://github.com/dotnet/roslyn/pull/69494))
14+
* Fix enter inserting /// on the incorrect line in documentation comments (PR: [#6130](https://github.com/dotnet/vscode-csharp/pull/6130))
15+
* Build extension with node 18 LTS (PR: [#6128](https://github.com/dotnet/vscode-csharp/pull/6128))
16+
* Update localized strings (PR: [#6129](https://github.com/dotnet/vscode-csharp/pull/6129))
17+
* Fix paths for package.nls.*.json (PR: [#6121](https://github.com/dotnet/vscode-csharp/pull/6121))
18+
* Add request to prepare for build diagnostic de-dupping (PR: [#6113](https://github.com/dotnet/vscode-csharp/pull/6113))
19+
* Support unit test debugging options in Roslyn LSP (PR: [#6110](https://github.com/dotnet/vscode-csharp/pull/6110))
20+
* Fix loading of package.nls.*.json (PR: [#6118](https://github.com/dotnet/vscode-csharp/pull/6118))
21+
* Add localization infrastructure to debugger package.json strings (PR: [#6088](https://github.com/dotnet/vscode-csharp/pull/6088))
22+
* Adjust C# semantic token scopes to better match 1.26 (PR: [#6094](https://github.com/dotnet/vscode-csharp/pull/6094))
23+
* Update Razor to 7.0.0-preview.23410.1 (PR: [#6105](https://github.com/dotnet/vscode-csharp/pull/6105))
24+
* Implement razor support for method simplification (PR: [#5982](https://github.com/dotnet/vscode-csharp/pull/5982))
25+
* Attempt to find a valid dotnet version from PATH before using runtime installer extension (PR: [#6074](https://github.com/dotnet/vscode-csharp/pull/6074))
26+
* Respect background analysis scope option in O# (PR: [#6058](https://github.com/dotnet/vscode-csharp/pull/6058))
27+
* Add localization infrastructure to debugger components (PR: [#6064](https://github.com/dotnet/vscode-csharp/pull/6064))
28+
* Add coreclr as a search keyword (PR: [#6071](https://github.com/dotnet/vscode-csharp/pull/6071))
29+
30+
## 2.0.357
1231
* Fix issue with Go to Definition giving a "unable to resolve reference" error (PR: [#69453](https://github.com/dotnet/roslyn/pull/69453))
1332
* Fix completion items not correctly adding using statements to the top of the file (PR: [#69454](https://github.com/dotnet/roslyn/pull/69454))
1433
* Improve de-duping of project load failure toasts (PR: [#69455](https://github.com/dotnet/roslyn/pull/69455))
@@ -25,6 +44,7 @@
2544
* Enable loading translated strings from razor TS code (PR: [#5962](https://github.com/dotnet/vscode-csharp/pull/5962))
2645
* Update typescript and eslint plugin to remove build warning (PR: [6002](https://github.com/dotnet/vscode-csharp/pull/6002))
2746
* Report how the extension was activated (PR: [#6043](https://github.com/dotnet/vscode-csharp/pull/6043))
47+
* Update Razor to 7.0.0-preview.23410.1 (PR: [#6105](https://github.com/dotnet/vscode-csharp/pull/6105))
2848

2949
## 2.0.328
3050
* Update Roslyn to 4.8.0-1.23403.6 (PR: [#6003](https://github.com/dotnet/vscode-csharp/pull/6003))

CONTRIBUTING.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,8 @@
44

55
First install:
66

7-
* Node.js ([v16.16 LTS](https://nodejs.org/en/blog/release/v16.16.0) is recommended).
8-
* Npm (The version shipped with node is fine, for example for v16.16.0 it would be 8.19.2)
7+
* Node.js ([v18.17.0 LTS](https://nodejs.org/de/blog/release/v18.17.0) is recommended).
8+
* Npm (The version shipped with node is fine)
99
* .NET 7.0 SDK (dotnet should be on your path)
1010

1111
### Build and run the extension

azure-pipelines-official.yml

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@ trigger:
22
branches:
33
include:
44
- main
5-
- prerelease
65
- release
76
pr: none
87

@@ -23,6 +22,6 @@ stages:
2322
prereleaseFlag: $(prereleaseFlag)
2423
pool:
2524
name: NetCore1ESPool-Internal
26-
demands: ImageOverride -equals Build.Ubuntu.1804.Amd64
25+
demands: ImageOverride -equals Build.Ubuntu.2204.Amd64
2726

2827
# TODO: add compliance, signing.

azure-pipelines.yml

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3,13 +3,11 @@ trigger:
33
branches:
44
include:
55
- feature/*
6-
- prerelease
76
- release
87
- main
98

109
pr:
1110
- feature/*
12-
- prerelease
1311
- release
1412
- main
1513

@@ -30,7 +28,7 @@ stages:
3028
prereleaseFlag: $(prereleaseFlag)
3129
pool:
3230
name: NetCore-Public
33-
demands: ImageOverride -equals Build.Ubuntu.1804.Amd64.Open
31+
demands: ImageOverride -equals Build.Ubuntu.2204.Amd64.Open
3432

3533
- stage: Test
3634
displayName: Test
@@ -40,7 +38,7 @@ stages:
4038
strategy:
4139
matrix:
4240
linux:
43-
demandsName: ImageOverride -equals Build.Ubuntu.1804.Amd64.Open
41+
demandsName: ImageOverride -equals Build.Ubuntu.2204.Amd64.Open
4442
pool:
4543
name: NetCore-Public
4644
demands: $(demandsName)

azure-pipelines/prereqs.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,14 +4,14 @@ steps:
44
- task: NuGetAuthenticate@1
55

66
- task: NodeTool@0
7-
displayName: 'Install Node.js 16.x'
7+
displayName: 'Install Node.js 18.x'
88
inputs:
9-
versionSpec: '16.x'
9+
versionSpec: '18.x'
1010

1111
- task: UseDotNet@2
1212
displayName: 'Install .NET Core SDKs'
1313
inputs:
14-
version: '7.0.100'
14+
version: '7.x'
1515

1616
# Set the CI build number to the VSIX version we're creating from this build.
1717
- script: |

azure-pipelines/release.yml

Lines changed: 12 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,16 @@
11
trigger: none
22
pr: none
33

4+
resources:
5+
pipelines:
6+
- pipeline: officialBuildCI
7+
source: dotnet-vscode-csharp
8+
branch: main
9+
410
parameters:
511
- name: test
612
type: boolean
713
default: true
8-
- name: branch
9-
type: string
10-
default: prerelease
11-
values:
12-
- prerelease
13-
- release
1414

1515
variables:
1616
# This is expected to provide VisualStudioMarketplacePAT to the release (https://code.visualstudio.com/api/working-with-extensions/publishing-extension#get-a-personal-access-token)
@@ -27,8 +27,9 @@ jobs:
2727
buildType: 'specific'
2828
project: 'internal'
2929
definition: 1264
30-
buildVersionToDownload: 'latestFromBranch'
31-
branchName: 'refs/heads/${{ parameters.branch }}'
30+
buildVersionToDownload: 'specific'
31+
buildId: '$(resources.pipeline.officialBuildCI.runID)'
32+
branchName: '$(resources.pipeline.officialBuildCI.sourceBranch)'
3233
- pwsh: |
3334
npm install --global vsce
3435
displayName: 'Install vsce'
@@ -44,13 +45,13 @@ jobs:
4445
4546
$additionalPublishArgs = , "publish"
4647
# Artifacts are published to either pre-release or release based on the build branch, https://code.visualstudio.com/api/working-with-extensions/publishing-extension#prerelease-extensions
47-
If ("${{ parameters.branch }}" -eq "prerelease") {
48+
If ("$(resources.pipeline.officialBuildCI.sourceBranch)" -eq "refs/heads/main") {
4849
$additionalPublishArgs += "--pre-release"
4950
Write-Host "Publish to pre-release channel."
50-
} ElseIf ("${{ parameters.branch }}" -eq "release") {
51+
} ElseIf ("$(resources.pipeline.officialBuildCI.sourceBranch)" -eq "refs/heads/release") {
5152
Write-Host "Publish to release channel."
5253
} Else {
53-
throw "Unexpected branch name: ${{ parameters.branch }}."
54+
throw "Unexpected branch name: $(resources.pipeline.officialBuildCI.sourceBranch)."
5455
}
5556
$additionalPublishArgs += '--packagePath'
5657

0 commit comments

Comments
 (0)