From 8d4363b515fdaa2918fdd8c5a2393af0d6b2ba80 Mon Sep 17 00:00:00 2001 From: "N. Taylor Mullen" Date: Wed, 16 May 2018 15:32:57 -0700 Subject: [PATCH 1/2] Update VSIX publishing to be consistent with non-VSIX publishing - Prior to this change PDBs published by our build system wouldn't match up because the assembly identity would look different. --- build/VSIX.targets | 2 ++ 1 file changed, 2 insertions(+) diff --git a/build/VSIX.targets b/build/VSIX.targets index febfc39d48..dcee315642 100644 --- a/build/VSIX.targets +++ b/build/VSIX.targets @@ -104,6 +104,8 @@ /p:TargetVSIXContainer=$(VSIXOutputPath); /p:SymbolsPublishDir=$(BuildDir); /p:Configuration=$(Configuration); + /p:FeatureBranchVersionSuffix=$(FeatureBranchVersionSuffix); + /p:BuildNumber=$(BuildNumber); /p:LangVersion=7.1" /> From e8c8396298c43b62647e9965bd1791401bc2f9cf Mon Sep 17 00:00:00 2001 From: "N. Taylor Mullen" Date: Wed, 16 May 2018 15:33:33 -0700 Subject: [PATCH 2/2] Upgrade to VS 15.8 Preview 2 version of Roslyn. --- build/dependencies.props | 20 +++++++++---------- .../RazorDirectiveCompletionProvider.cs | 3 ++- .../RazorDirectiveCompletionProviderTest.cs | 3 ++- 3 files changed, 14 insertions(+), 12 deletions(-) diff --git a/build/dependencies.props b/build/dependencies.props index c5fa7fdcc1..ad3541cf06 100644 --- a/build/dependencies.props +++ b/build/dependencies.props @@ -45,16 +45,16 @@ 4.5.0-preview3-26331-02 4.5.0-preview3-26331-02 9.0.1 - 2.7.0-beta3-62512-06 - 2.7.0-beta3-62512-06 - 2.7.0-beta3-62512-06 - 2.7.0-beta3-62512-06 - 2.7.0-beta3-62512-06 - 2.7.0-beta3-62512-06 - 2.7.0-beta3-62512-06 - 2.7.0-beta3-62512-06 - 2.7.0-beta3-62512-06 - 2.7.0-beta3-62512-06 + 2.9.0-beta4-62911-02 + 2.9.0-beta4-62911-02 + 2.9.0-beta4-62911-02 + 2.9.0-beta4-62911-02 + 2.9.0-beta4-62911-02 + 2.9.0-beta4-62911-02 + 2.9.0-beta4-62911-02 + 2.9.0-beta4-62911-02 + 2.9.0-beta4-62911-02 + 2.9.0-beta4-62911-02 0.8.0 2.3.1 2.4.0-beta.1.build3945 diff --git a/src/Microsoft.VisualStudio.Editor.Razor/RazorDirectiveCompletionProvider.cs b/src/Microsoft.VisualStudio.Editor.Razor/RazorDirectiveCompletionProvider.cs index c55dcc3776..56768829d1 100644 --- a/src/Microsoft.VisualStudio.Editor.Razor/RazorDirectiveCompletionProvider.cs +++ b/src/Microsoft.VisualStudio.Editor.Razor/RazorDirectiveCompletionProvider.cs @@ -14,6 +14,7 @@ using Microsoft.AspNetCore.Razor.Language.Legacy; using Microsoft.CodeAnalysis; using Microsoft.CodeAnalysis.Completion; using Microsoft.CodeAnalysis.Razor; +using Microsoft.CodeAnalysis.Tags; using Microsoft.CodeAnalysis.Text; using Microsoft.VisualStudio.Text; using Microsoft.VisualStudio.Text.Projection; @@ -139,7 +140,7 @@ namespace Microsoft.VisualStudio.Editor.Razor // This groups all Razor directives together sortText: "_RazorDirective_", rules: CompletionItemRules.Create(formatOnCommit: false), - tags: ImmutableArray.Create(CompletionTags.Intrinsic), + tags: ImmutableArray.Create(WellKnownTags.Intrinsic), properties: propertyDictionary.ToImmutableDictionary()); completionItems.Add(completionItem); } diff --git a/test/Microsoft.VisualStudio.Editor.Razor.Test/RazorDirectiveCompletionProviderTest.cs b/test/Microsoft.VisualStudio.Editor.Razor.Test/RazorDirectiveCompletionProviderTest.cs index 0b8ccafb91..1b76f772b6 100644 --- a/test/Microsoft.VisualStudio.Editor.Razor.Test/RazorDirectiveCompletionProviderTest.cs +++ b/test/Microsoft.VisualStudio.Editor.Razor.Test/RazorDirectiveCompletionProviderTest.cs @@ -14,6 +14,7 @@ using Microsoft.AspNetCore.Razor.Language.Legacy; using Microsoft.CodeAnalysis; using Microsoft.CodeAnalysis.Completion; using Microsoft.CodeAnalysis.Options; +using Microsoft.CodeAnalysis.Tags; using Microsoft.CodeAnalysis.Text; using Microsoft.VisualStudio.Text; using Moq; @@ -289,7 +290,7 @@ namespace Microsoft.VisualStudio.Editor.Razor Assert.Equal("_RazorDirective_", item.SortText); Assert.False(item.Rules.FormatOnCommit); var tag = Assert.Single(item.Tags); - Assert.Equal(CompletionTags.Intrinsic, tag); + Assert.Equal(WellKnownTags.Intrinsic, tag); } private static Lazy CreateCodeDocumentProvider(string text, IEnumerable directives)