Merge branch 'master' into azure-pipelines/certificate-sha1

Author: jcansdale

.github/workflows/main.yml

@@ -0,0 +1,77 @@
+# This is a basic workflow to help you get started with Actions
+
+name: CI
+
+env:
+  config: Release
+  githubvsSolution: GitHubVS.sln
+  vsixContainer: ${{ github.workspace }}\GitHub.VisualStudio.vsix
+  
+# Controls when the action will run. Triggers the workflow on push or pull request 
+# events but only for the master branch
+on:
+  push:
+    branches: [ master ]
+  pull_request:
+    branches: [ master ]
+
+# A workflow run is made up of one or more jobs that can run sequentially or in parallel
+jobs:
+  # This workflow contains a single job called "build"
+  build:
+    # The type of runner that the job will run on
+    runs-on: windows-latest
+
+    # Steps represent a sequence of tasks that will be executed as part of the job
+    steps:
+    # Checks-out your repository under $GITHUB_WORKSPACE, so your job can access it
+    - uses: actions/checkout@v1
+      with:
+        submodules: true
+
+    - name: Nerdbank.GitVersioning
+      uses: aarnott/[email protected]
+      id: nbgv
+
+    - name: Add MSBuild to PATH
+      uses: microsoft/[email protected]
+
+    - uses: nuget/setup-nuget@v1
+      with:
+        nuget-version: '5.x'
+
+    - name: Add VSTest to PATH
+      uses: darenm/Setup-VSTest@v1
+
+    - name: NuGet restore ${{ env.githubvsSolution }}
+      run: nuget restore ${{ env.githubvsSolution }}
+
+    - name: MSBuild ${{ env.githubvsSolution }}
+      run: |
+        msbuild ${{ env.githubvsSolution }} /p:Configuration=${{ env.config }} /p:TargetVsixContainer=${{ env.vsixContainer }} /p:DeployExtension=False /verbosity:minimal
+      env:
+        GitHubVS_ClientId: ${{ secrets.GitHubVS_ClientId }}
+        GitHubVS_ClientSecret: ${{ secrets.GitHubVS_ClientSecret }}
+
+    - name: Upload VSIX artifact
+      uses: actions/upload-artifact@v1
+      with:
+        name: GitHubVS-${{ steps.nbgv.outputs.SemVer2 }}
+        path: ${{ env.vsixContainer }}
+
+    # We need to run '**\bin\**\*Tests.dll'
+    - name: Run unit tests
+      shell: bash
+      run: |
+        vstest.console /TestAdapterPath:test /Settings:test/test.runsettings \
+          test/GitHub.App.UnitTests/bin/${{ env.config }}/net46/GitHub.App.UnitTests.dll \
+          test/GitHub.Exports.Reactive.UnitTests/bin/${{ env.config }}/net46/GitHub.Exports.Reactive.UnitTests.dll \
+          test/GitHub.Exports.UnitTests/bin/${{ env.config }}/net46/GitHub.Exports.UnitTests.dll \
+          test/GitHub.Extensions.UnitTests/bin/${{ env.config }}/net46/GitHub.Extensions.UnitTests.dll \
+          test/GitHub.InlineReviews.UnitTests/bin/${{ env.config }}/net46/GitHub.InlineReviews.UnitTests.dll \
+          test/GitHub.Services.Vssdk.UnitTests/bin/${{ env.config }}/net461/GitHub.Services.Vssdk.UnitTests.dll \
+          test/GitHub.StartPage.UnitTests/bin/${{ env.config }}/net46/GitHub.StartPage.UnitTests.dll \
+          test/GitHub.TeamFoundation.UnitTests/bin/${{ env.config }}/net46/GitHub.TeamFoundation.UnitTests.dll \
+          test/GitHub.UI.UnitTests/bin/${{ env.config }}/net46/GitHub.UI.UnitTests.dll \
+          test/GitHub.VisualStudio.UnitTests/bin/${{ env.config }}/net46/GitHub.VisualStudio.UnitTests.dll \
+          test/MetricsTests/MetricsTests/bin/${{ env.config }}/MetricsTests.dll

README.md

@@ -7,7 +7,7 @@ Most of the extension UI lives in the Team Explorer pane, which is available fro
 
 Official builds of this extension are available at the [Visual Studio Marketplace](https://marketplace.visualstudio.com/items?itemName=GitHub.GitHubExtensionforVisualStudio).
 
-[![Build Status](https://github-editor-tools.visualstudio.com/VisualStudio/_apis/build/status/github.VisualStudio?branchName=master)](https://github-editor-tools.visualstudio.com/VisualStudio/_build/latest?definitionId=10&branchName=master)
+![CI](https://github.com/github/visualstudio/workflows/CI/badge.svg)
 
 [![Follow GitHub for Visual Studio](https://img.shields.io/twitter/follow/GitHubVS.svg?style=social "Follow GitHubVS")](https://twitter.com/githubvs?ref_src=twsrc%5Etfw) [![Join the chat at https://gitter.im/github/VisualStudio](https://badges.gitter.im/Join%20Chat.svg)](https://gitter.im/github/VisualStudio?utm_source=badge&utm_medium=badge&utm_campaign=pr-badge&utm_content=badge)
 
@@ -72,4 +72,3 @@ Visit the [Contributor Guidelines](CONTRIBUTING.md) for details on how to contri
 Copyright 2015 - 2019 GitHub, Inc.
 
 Licensed under the [MIT License](LICENSE.md)
-

src/GitHub.Api/ApiClientConfiguration.cs

@@ -40,7 +40,7 @@ static ApiClientConfiguration()
         /// <summary>
         /// Gets the ideal scopes requested by the application.
         /// </summary>
-        public static IReadOnlyList<string> RequestedScopes { get; } = new[] { "user", "repo", "gist", "write:public_key", "read:org" };
+        public static IReadOnlyList<string> RequestedScopes { get; } = new[] { "user", "repo", "gist", "write:public_key", "read:org", "workflow" };
 
         /// <summary>
         /// Gets a note that will be stored with the OAUTH token.

src/GitHub.Api/GitHub.Api.csproj

@@ -4,7 +4,14 @@
     <DebugType>full</DebugType>
     <DebugSymbols>true</DebugSymbols>
   </PropertyGroup>
-  
+   
+  <PropertyGroup>
+    <!-- Use a default client ID/secret for testing -->
+    <!-- https://github.com/organizations/editor-tools/settings/applications/620133 -->
+    <GitHubVS_ClientId Condition="'$(GitHubVS_ClientId)' == ''">2454a3e6102fd41cc212</GitHubVS_ClientId>
+    <GitHubVS_ClientSecret Condition="'$(GitHubVS_ClientSecret)' == ''">2157c138e970165d955d09562230afcfbcda23f2</GitHubVS_ClientSecret>
+  </PropertyGroup>
+   
   <Import Project="$(SolutionDir)\src\common\signing.props" />
 
   <Target Name="AddClientSecret" BeforeTargets="BeforeCompile"

src/GitHub.App/Services/FromGraphQlExtensions.cs

@@ -29,6 +29,10 @@ public static class FromGraphQlExtensions
                     return CheckConclusionState.Success;
                 case Octokit.GraphQL.Model.CheckConclusionState.Neutral:
                     return CheckConclusionState.Neutral;
+                case Octokit.GraphQL.Model.CheckConclusionState.Skipped:
+                    return CheckConclusionState.Skipped;
+                case Octokit.GraphQL.Model.CheckConclusionState.Stale:
+                    return CheckConclusionState.Stale;
                 default:
                     throw new ArgumentOutOfRangeException(nameof(value), value, null);
             }

src/GitHub.App/Services/MentionsAutoCompleteSource.cs

@@ -54,7 +54,7 @@ public IObservable<AutoCompleteSuggestion> GetSuggestions()
             {
                 query = new Query().Repository(owner: Var(nameof(owner)), name: Var(nameof(name)))
                 .Select(repository =>
-                    repository.MentionableUsers(null, null, null, null)
+                    repository.MentionableUsers(null, null, null, null, null)
                         .AllPages()
                         .Select(sourceItem => 
                             new SuggestionItem(sourceItem.Login, 

src/GitHub.App/ViewModels/Dialog/LoginCredentialsViewModel.cs

@@ -1,8 +1,6 @@
 using System;
 using System.ComponentModel.Composition;
 using System.Reactive.Linq;
-using GitHub.App;
-using GitHub.Primitives;
 using GitHub.Services;
 using ReactiveUI;
 
@@ -28,8 +26,7 @@ public LoginCredentialsViewModel(
                 (x, y) => x.Value || y.Value
             ).ToProperty(this, vm => vm.IsLoginInProgress);
 
-            UpdateLoginMode();
-            connectionManager.Connections.CollectionChanged += (_, __) => UpdateLoginMode();
+            LoginMode = LoginMode.DotComOrEnterprise;
 
             Done = Observable.Merge(
                 loginToGitHubViewModel.Login,
@@ -55,21 +52,5 @@ public LoginMode LoginMode
         public bool IsLoginInProgress { get { return isLoginInProgress.Value; } }
 
         public IObservable<object> Done { get; }
-
-        void UpdateLoginMode()
-        {
-            var result = LoginMode.DotComOrEnterprise;
-
-            foreach (var connection in ConnectionManager.Connections)
-            {
-                if (connection.IsLoggedIn)
-                {
-                    result &= ~((connection.HostAddress == HostAddress.GitHubDotComHostAddress) ?
-                        LoginMode.DotComOnly : LoginMode.EnterpriseOnly);
-                }
-            }
-
-            LoginMode = result;
-        }
     }
 }

src/GitHub.App/ViewModels/GitHubPane/GitHubPaneViewModel.cs

@@ -160,6 +160,14 @@ public GitHubPaneViewModel(
                 },
                 currentPage.Select(x => x is IOpenInBrowser));
 
+            BrowseRepository = ReactiveCommand.Create(
+                () =>
+                {
+                    var url = LocalRepository.CloneUrl.ToRepositoryUrl();
+                    if (url != null) browser.OpenUrl(url);
+                },
+                currentPage.Select(x => x is IOpenInBrowser));
+
             help = ReactiveCommand.Create(() => { });
             help.Subscribe(_ =>
             {
@@ -531,5 +539,7 @@ static Regex CreateRoute(string route)
             var routeFormat = "^" + new Regex("(:([a-z]+))\\b").Replace(route, @"(?<$2>[\w_.\-=]+)") + "$";
             return new Regex(routeFormat, RegexOptions.ExplicitCapture | RegexOptions.IgnoreCase);
         }
+
+        public ReactiveCommand<Unit, Unit> BrowseRepository { get; }
     }
 }

src/GitHub.Exports/Models/CheckConclusionState.cs

@@ -8,5 +8,7 @@ public enum CheckConclusionState
         Failure,
         Success,
         Neutral,
+        Skipped,
+        Stale
     }
 }

src/GitHub.Exports/Settings/generated/IPackageSettings.cs

@@ -7,11 +7,6 @@
       "type": "bool",
       "default": 'true'
     },
-    {
-      "name": "EditorComments",
-      "type": "bool",
-      "default": "false"
-    },
     {
       "name": "UIState",
       "type": "object",
@@ -40,7 +35,6 @@ public interface IPackageSettings : INotifyPropertyChanged
     {
         void Save();
         bool CollectMetrics { get; set; }
-        bool EditorComments { get; set; }
         UIState UIState { get; set; }
         bool HideTeamExplorerWelcomeMessage { get; set; }
         bool EnableTraceLogging { get; set; }