├── .gitattributes ├── .gitignore ├── ApiReviewList.sln ├── LICENSE ├── README.md ├── build.cmd ├── docs └── screen.png └── src ├── ApiKeyStore.cs ├── ApiReviewList.csproj ├── App.config ├── App.xaml ├── App.xaml.cs ├── GitHubClientFactory.cs ├── GitHubIssueHelpers.cs ├── MainWindow.xaml ├── MainWindow.xaml.cs ├── NotesDialog.xaml ├── NotesDialog.xaml.cs ├── OrgAndRepo.cs ├── Properties ├── AssemblyInfo.cs ├── Resources.Designer.cs ├── Resources.resx ├── Settings.Designer.cs └── Settings.settings ├── Reports ├── ApiReviewFeedback.cs ├── ApiReviewFeedbackWithVideo.cs ├── ApiReviewSummary.cs └── ApiReviewVideo.cs ├── Shell.cs ├── ViewModels ├── Command.cs ├── IssueListViewModel.cs ├── IssueViewModel.cs ├── LabelViewModel.cs ├── MilestoneViewModel.cs ├── NotesEntryViewModel.cs ├── NotesViewModel.cs ├── SendOptionsViewModel.cs ├── TimeFormatting.cs └── ViewModel.cs ├── YouTubeKeyStore.cs ├── YouTubeServiceFactory.cs ├── app.manifest └── list.ico /.gitattributes: -------------------------------------------------------------------------------- 1 | ############################################################################### 2 | # Set default behavior to automatically normalize line endings. 3 | ############################################################################### 4 | * text=auto 5 | 6 | ############################################################################### 7 | # Set default behavior for command prompt diff. 8 | # 9 | # This is need for earlier builds of msysgit that does not have it on by 10 | # default for csharp files. 11 | # Note: This is only used by command line 12 | ############################################################################### 13 | #*.cs diff=csharp 14 | 15 | ############################################################################### 16 | # Set the merge driver for project and solution files 17 | # 18 | # Merging from the command prompt will add diff markers to the files if there 19 | # are conflicts (Merging from VS is not affected by the settings below, in VS 20 | # the diff markers are never inserted). Diff markers may cause the following 21 | # file extensions to fail to load in VS. An alternative would be to treat 22 | # these files as binary and thus will always conflict and require user 23 | # intervention with every merge. To do so, just uncomment the entries below 24 | ############################################################################### 25 | #*.sln merge=binary 26 | #*.csproj merge=binary 27 | #*.vbproj merge=binary 28 | #*.vcxproj merge=binary 29 | #*.vcproj merge=binary 30 | #*.dbproj merge=binary 31 | #*.fsproj merge=binary 32 | #*.lsproj merge=binary 33 | #*.wixproj merge=binary 34 | #*.modelproj merge=binary 35 | #*.sqlproj merge=binary 36 | #*.wwaproj merge=binary 37 | 38 | ############################################################################### 39 | # behavior for image files 40 | # 41 | # image files are treated as binary by default. 42 | ############################################################################### 43 | #*.jpg binary 44 | #*.png binary 45 | #*.gif binary 46 | 47 | ############################################################################### 48 | # diff behavior for common document formats 49 | # 50 | # Convert binary document formats to text before diffing them. This feature 51 | # is only available from the command line. Turn it on by uncommenting the 52 | # entries below. 53 | ############################################################################### 54 | #*.doc diff=astextplain 55 | #*.DOC diff=astextplain 56 | #*.docx diff=astextplain 57 | #*.DOCX diff=astextplain 58 | #*.dot diff=astextplain 59 | #*.DOT diff=astextplain 60 | #*.pdf diff=astextplain 61 | #*.PDF diff=astextplain 62 | #*.rtf diff=astextplain 63 | #*.RTF diff=astextplain 64 | -------------------------------------------------------------------------------- /.gitignore: -------------------------------------------------------------------------------- 1 | ## Ignore Visual Studio temporary files, build results, and 2 | ## files generated by popular Visual Studio add-ons. 3 | 4 | # User-specific files 5 | *.suo 6 | *.user 7 | *.userosscache 8 | *.sln.docstates 9 | 10 | # User-specific files (MonoDevelop/Xamarin Studio) 11 | *.userprefs 12 | 13 | # Build results 14 | [Dd]ebug/ 15 | [Dd]ebugPublic/ 16 | [Rr]elease/ 17 | [Rr]eleases/ 18 | x64/ 19 | x86/ 20 | bld/ 21 | [Bb]in/ 22 | [Oo]bj/ 23 | [Ll]og/ 24 | 25 | # Visual Studio 2015 cache/options directory 26 | .vs/ 27 | # Uncomment if you have tasks that create the project's static files in wwwroot 28 | #wwwroot/ 29 | 30 | # MSTest test Results 31 | [Tt]est[Rr]esult*/ 32 | [Bb]uild[Ll]og.* 33 | 34 | # NUNIT 35 | *.VisualState.xml 36 | TestResult.xml 37 | 38 | # Build Results of an ATL Project 39 | [Dd]ebugPS/ 40 | [Rr]eleasePS/ 41 | dlldata.c 42 | 43 | # DNX 44 | project.lock.json 45 | project.fragment.lock.json 46 | artifacts/ 47 | 48 | *_i.c 49 | *_p.c 50 | *_i.h 51 | *.ilk 52 | *.meta 53 | *.obj 54 | *.pch 55 | *.pdb 56 | *.pgc 57 | *.pgd 58 | *.rsp 59 | *.sbr 60 | *.tlb 61 | *.tli 62 | *.tlh 63 | *.tmp 64 | *.tmp_proj 65 | *.log 66 | *.vspscc 67 | *.vssscc 68 | .builds 69 | *.pidb 70 | *.svclog 71 | *.scc 72 | 73 | # Chutzpah Test files 74 | _Chutzpah* 75 | 76 | # Visual C++ cache files 77 | ipch/ 78 | *.aps 79 | *.ncb 80 | *.opendb 81 | *.opensdf 82 | *.sdf 83 | *.cachefile 84 | *.VC.db 85 | *.VC.VC.opendb 86 | 87 | # Visual Studio profiler 88 | *.psess 89 | *.vsp 90 | *.vspx 91 | *.sap 92 | 93 | # TFS 2012 Local Workspace 94 | $tf/ 95 | 96 | # Guidance Automation Toolkit 97 | *.gpState 98 | 99 | # ReSharper is a .NET coding add-in 100 | _ReSharper*/ 101 | *.[Rr]e[Ss]harper 102 | *.DotSettings.user 103 | 104 | # JustCode is a .NET coding add-in 105 | .JustCode 106 | 107 | # TeamCity is a build add-in 108 | _TeamCity* 109 | 110 | # DotCover is a Code Coverage Tool 111 | *.dotCover 112 | 113 | # NCrunch 114 | _NCrunch_* 115 | .*crunch*.local.xml 116 | nCrunchTemp_* 117 | 118 | # MightyMoose 119 | *.mm.* 120 | AutoTest.Net/ 121 | 122 | # Web workbench (sass) 123 | .sass-cache/ 124 | 125 | # Installshield output folder 126 | [Ee]xpress/ 127 | 128 | # DocProject is a documentation generator add-in 129 | DocProject/buildhelp/ 130 | DocProject/Help/*.HxT 131 | DocProject/Help/*.HxC 132 | DocProject/Help/*.hhc 133 | DocProject/Help/*.hhk 134 | DocProject/Help/*.hhp 135 | DocProject/Help/Html2 136 | DocProject/Help/html 137 | 138 | # Click-Once directory 139 | publish/ 140 | 141 | # Publish Web Output 142 | *.[Pp]ublish.xml 143 | *.azurePubxml 144 | # TODO: Comment the next line if you want to checkin your web deploy settings 145 | # but database connection strings (with potential passwords) will be unencrypted 146 | #*.pubxml 147 | *.publishproj 148 | 149 | # Microsoft Azure Web App publish settings. Comment the next line if you want to 150 | # checkin your Azure Web App publish settings, but sensitive information contained 151 | # in these scripts will be unencrypted 152 | PublishScripts/ 153 | 154 | # NuGet Packages 155 | *.nupkg 156 | # The packages folder can be ignored because of Package Restore 157 | **/packages/* 158 | # except build/, which is used as an MSBuild target. 159 | !**/packages/build/ 160 | # Uncomment if necessary however generally it will be regenerated when needed 161 | #!**/packages/repositories.config 162 | # NuGet v3's project.json files produces more ignoreable files 163 | *.nuget.props 164 | *.nuget.targets 165 | 166 | # Microsoft Azure Build Output 167 | csx/ 168 | *.build.csdef 169 | 170 | # Microsoft Azure Emulator 171 | ecf/ 172 | rcf/ 173 | 174 | # Windows Store app package directories and files 175 | AppPackages/ 176 | BundleArtifacts/ 177 | Package.StoreAssociation.xml 178 | _pkginfo.txt 179 | 180 | # Visual Studio cache files 181 | # files ending in .cache can be ignored 182 | *.[Cc]ache 183 | # but keep track of directories ending in .cache 184 | !*.[Cc]ache/ 185 | 186 | # Others 187 | ClientBin/ 188 | ~$* 189 | *~ 190 | *.dbmdl 191 | *.dbproj.schemaview 192 | *.jfm 193 | *.pfx 194 | *.publishsettings 195 | node_modules/ 196 | orleans.codegen.cs 197 | 198 | # Since there are multiple workflows, uncomment next line to ignore bower_components 199 | # (https://github.com/github/gitignore/pull/1529#issuecomment-104372622) 200 | #bower_components/ 201 | 202 | # RIA/Silverlight projects 203 | Generated_Code/ 204 | 205 | # Backup & report files from converting an old project file 206 | # to a newer Visual Studio version. Backup files are not needed, 207 | # because we have git ;-) 208 | _UpgradeReport_Files/ 209 | Backup*/ 210 | UpgradeLog*.XML 211 | UpgradeLog*.htm 212 | 213 | # SQL Server files 214 | *.mdf 215 | *.ldf 216 | 217 | # Business Intelligence projects 218 | *.rdl.data 219 | *.bim.layout 220 | *.bim_*.settings 221 | 222 | # Microsoft Fakes 223 | FakesAssemblies/ 224 | 225 | # GhostDoc plugin setting file 226 | *.GhostDoc.xml 227 | 228 | # Node.js Tools for Visual Studio 229 | .ntvs_analysis.dat 230 | 231 | # Visual Studio 6 build log 232 | *.plg 233 | 234 | # Visual Studio 6 workspace options file 235 | *.opt 236 | 237 | # Visual Studio LightSwitch build output 238 | **/*.HTMLClient/GeneratedArtifacts 239 | **/*.DesktopClient/GeneratedArtifacts 240 | **/*.DesktopClient/ModelManifest.xml 241 | **/*.Server/GeneratedArtifacts 242 | **/*.Server/ModelManifest.xml 243 | _Pvt_Extensions 244 | 245 | # Paket dependency manager 246 | .paket/paket.exe 247 | paket-files/ 248 | 249 | # FAKE - F# Make 250 | .fake/ 251 | 252 | # JetBrains Rider 253 | .idea/ 254 | *.sln.iml 255 | 256 | # CodeRush 257 | .cr/ 258 | 259 | # Python Tools for Visual Studio (PTVS) 260 | __pycache__/ 261 | *.pyc -------------------------------------------------------------------------------- /ApiReviewList.sln: -------------------------------------------------------------------------------- 1 |  2 | Microsoft Visual Studio Solution File, Format Version 12.00 3 | # Visual Studio Version 16 4 | VisualStudioVersion = 16.0.28411.128 5 | MinimumVisualStudioVersion = 10.0.40219.1 6 | Project("{FAE04EC0-301F-11D3-BF4B-00C04F79EFBC}") = "ApiReviewList", "src\ApiReviewList.csproj", "{0E0CF68D-087A-4DBC-8EF9-39F206FC99EF}" 7 | EndProject 8 | Global 9 | GlobalSection(SolutionConfigurationPlatforms) = preSolution 10 | Debug|Any CPU = Debug|Any CPU 11 | Release|Any CPU = Release|Any CPU 12 | EndGlobalSection 13 | GlobalSection(ProjectConfigurationPlatforms) = postSolution 14 | {0E0CF68D-087A-4DBC-8EF9-39F206FC99EF}.Debug|Any CPU.ActiveCfg = Debug|Any CPU 15 | {0E0CF68D-087A-4DBC-8EF9-39F206FC99EF}.Debug|Any CPU.Build.0 = Debug|Any CPU 16 | {0E0CF68D-087A-4DBC-8EF9-39F206FC99EF}.Release|Any CPU.ActiveCfg = Release|Any CPU 17 | {0E0CF68D-087A-4DBC-8EF9-39F206FC99EF}.Release|Any CPU.Build.0 = Release|Any CPU 18 | EndGlobalSection 19 | GlobalSection(SolutionProperties) = preSolution 20 | HideSolutionNode = FALSE 21 | EndGlobalSection 22 | GlobalSection(ExtensibilityGlobals) = postSolution 23 | SolutionGuid = {748E2293-DF4D-4F52-93B9-69410D550C7E} 24 | EndGlobalSection 25 | EndGlobal 26 | -------------------------------------------------------------------------------- /LICENSE: -------------------------------------------------------------------------------- 1 | The MIT License (MIT) 2 | 3 | Copyright (c) Immo Landwerth 4 | 5 | All rights reserved. 6 | 7 | Permission is hereby granted, free of charge, to any person obtaining a copy 8 | of this software and associated documentation files (the "Software"), to deal 9 | in the Software without restriction, including without limitation the rights 10 | to use, copy, modify, merge, publish, distribute, sublicense, and/or sell 11 | copies of the Software, and to permit persons to whom the Software is 12 | furnished to do so, subject to the following conditions: 13 | 14 | The above copyright notice and this permission notice shall be included in all 15 | copies or substantial portions of the Software. 16 | 17 | THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR 18 | IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, 19 | FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE 20 | AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER 21 | LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, 22 | OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE 23 | SOFTWARE. 24 | -------------------------------------------------------------------------------- /README.md: -------------------------------------------------------------------------------- 1 | # API Review List 2 | 3 | This tool browses GitHub repos and looks for issues that are ready for API 4 | review. 5 | 6 | ![](docs/screen.png) -------------------------------------------------------------------------------- /build.cmd: -------------------------------------------------------------------------------- 1 | @echo off 2 | setlocal 3 | 4 | :: Use VSINSTALLDIR if present 5 | 6 | if not "%VSINSTALLDIR%" == "" goto setmsbuild 7 | 8 | :: Use vswhere to find an install of VS 2017 9 | 10 | set VSWHERE="%ProgramFiles(x86)%\Microsoft Visual Studio\Installer\vswhere.exe" 11 | if not exist %VSWHERE% goto error 12 | for /f "tokens=*" %%i in ('%VSWHERE% -property installationPath -prerelease') do set VSINSTALLDIR=%%i 13 | 14 | :setmsbuild 15 | 16 | set MSBuild="%VSINSTALLDIR%\MSBuild\Current\Bin\MSBuild.exe" 17 | echo %MSBuild% 18 | if exist %MSBuild% goto build 19 | 20 | :error 21 | 22 | if exist %MSBuild% goto build 23 | echo ERROR: You need Visual Studio to build. 24 | exit /B -1 25 | 26 | :build 27 | 28 | setlocal 29 | set OutDir="%~dp0bin" 30 | 31 | if not exist %OutDir% mkdir %OutDir% 32 | pushd src 33 | %MSBuild% /nologo /m /v:m /nr:false /bl:%OutDir%\msbuild.binlog /t:Restore /t:Build /p:OutDir=%OutDir% %* 34 | popd 35 | -------------------------------------------------------------------------------- /docs/screen.png: -------------------------------------------------------------------------------- https://raw.githubusercontent.com/terrajobst/api-review-list/6b2f117829adf40f699b53809f106606b7e483b6/docs/screen.png -------------------------------------------------------------------------------- /src/ApiKeyStore.cs: -------------------------------------------------------------------------------- 1 | using System; 2 | using System.IO; 3 | 4 | namespace ApiReviewList 5 | { 6 | internal static class ApiKeyStore 7 | { 8 | private static string GetPath() 9 | { 10 | var appData = Environment.GetFolderPath(Environment.SpecialFolder.ApplicationData); 11 | return Path.Combine(appData, "Microsoft", "APIReviewList", "apikey.txt"); 12 | } 13 | 14 | public static string GetApiKey() 15 | { 16 | var file = GetPath(); 17 | if (!File.Exists(file)) 18 | return null; 19 | 20 | return File.ReadAllText(file).Trim(); 21 | } 22 | 23 | public static void SetApiKey(string key) 24 | { 25 | var file = GetPath(); 26 | var directory = Path.GetDirectoryName(file); 27 | Directory.CreateDirectory(directory); 28 | File.WriteAllText(file, key); 29 | } 30 | } 31 | } 32 | -------------------------------------------------------------------------------- /src/ApiReviewList.csproj: -------------------------------------------------------------------------------- 1 |  2 | 3 | 4 | netcoreapp3.1 5 | True 6 | false 7 | WinExe 8 | app.manifest 9 | list.ico 10 | 11 | 12 | 13 | 14 | 00062fff-0000-0000-c000-000000000046 15 | 9 16 | 6 17 | tlbimp 18 | 0 19 | false 20 | true 21 | 22 | 23 | 24 | 25 | 26 | 27 | 28 | 29 | 30 | 31 | -------------------------------------------------------------------------------- /src/App.config: -------------------------------------------------------------------------------- 1 |  2 | 3 | 4 | 5 | 6 | -------------------------------------------------------------------------------- /src/App.xaml: -------------------------------------------------------------------------------- 1 |  5 | 6 | 7 | 8 | 9 | -------------------------------------------------------------------------------- /src/App.xaml.cs: -------------------------------------------------------------------------------- 1 | using System; 2 | using System.Threading.Tasks; 3 | using System.Windows; 4 | 5 | namespace ApiReviewList 6 | { 7 | public partial class App : Application 8 | { 9 | protected override async void OnStartup(StartupEventArgs e) 10 | { 11 | base.OnStartup(e); 12 | 13 | var apiKey = ApiKeyStore.GetApiKey(); 14 | 15 | if (string.IsNullOrEmpty(apiKey)) 16 | { 17 | var needKeyMessage = "In order to use this app, you'll need to register an API Key. Do you want do do this now?"; 18 | if (MessageBox.Show(needKeyMessage, "API Review List", MessageBoxButton.YesNo, MessageBoxImage.Information) == MessageBoxResult.No) 19 | Environment.Exit(1); 20 | 21 | await SetApiKey(); 22 | } 23 | else if (!await GitHubClientFactory.IsValidKeyAsync(apiKey)) 24 | { 25 | var needKeyMessage = "The API key is no longer valid. In order to use this app, you'll need to register a new API Key. Do you want do do this now?"; 26 | if (MessageBox.Show(needKeyMessage, "API Review List", MessageBoxButton.YesNo, MessageBoxImage.Information) == MessageBoxResult.No) 27 | Environment.Exit(1); 28 | 29 | await SetApiKey(); 30 | } 31 | 32 | var window = new MainWindow(); 33 | window.Show(); 34 | } 35 | 36 | private static async Task SetApiKey() 37 | { 38 | var registerKeyMessage = "Next, you'll need to go to GitHub and register a personal access token. It only needs public_repo access. Copy the token to the clipboard."; 39 | MessageBox.Show(registerKeyMessage, "API Review List", MessageBoxButton.OK, MessageBoxImage.Information); 40 | 41 | var url = "https://github.com/settings/tokens/new"; 42 | Shell.Execute(url); 43 | 44 | while (true) 45 | { 46 | var valiateTokenMessage = "Once the token is in the clipboard, click OK."; 47 | MessageBox.Show(valiateTokenMessage, "API Review List", MessageBoxButton.OK, MessageBoxImage.Information); 48 | 49 | var key = Clipboard.GetText(); 50 | if (await GitHubClientFactory.IsValidKeyAsync(key)) 51 | { 52 | ApiKeyStore.SetApiKey(key); 53 | break; 54 | } 55 | 56 | var keyIsInvalid = "The key in the clipboard is not valid."; 57 | MessageBox.Show(keyIsInvalid, "API Review List", MessageBoxButton.OK, MessageBoxImage.Error); 58 | } 59 | } 60 | } 61 | } 62 | -------------------------------------------------------------------------------- /src/GitHubClientFactory.cs: -------------------------------------------------------------------------------- 1 | using System; 2 | using System.Threading.Tasks; 3 | 4 | using Octokit; 5 | 6 | namespace ApiReviewList 7 | { 8 | internal static class GitHubClientFactory 9 | { 10 | public static GitHubClient Create() 11 | { 12 | var key = ApiKeyStore.GetApiKey(); 13 | return Create(key); 14 | } 15 | 16 | public static GitHubClient Create(string apiKey) 17 | { 18 | var productInformation = new ProductHeaderValue("APIReviewList"); 19 | var client = new GitHubClient(productInformation) 20 | { 21 | Credentials = new Credentials(apiKey) 22 | }; 23 | return client; 24 | } 25 | 26 | public static async Task IsValidKeyAsync(string apiKey) 27 | { 28 | try 29 | { 30 | var client = Create(apiKey); 31 | var request = new IssueRequest 32 | { 33 | Since = DateTimeOffset.Now 34 | }; 35 | await client.Issue.GetAllForCurrent(request); 36 | } 37 | catch (Exception) 38 | { 39 | return false; 40 | } 41 | 42 | return true; 43 | } 44 | } 45 | } 46 | -------------------------------------------------------------------------------- /src/GitHubIssueHelpers.cs: -------------------------------------------------------------------------------- 1 | using System; 2 | 3 | namespace ApiReviewList 4 | { 5 | internal static class GitHubIssueHelpers 6 | { 7 | public static string FixTitle(string title) 8 | { 9 | var prefixes = new[] 10 | { 11 | "api proposal", 12 | "[api proposal]", 13 | "api", 14 | "[api]", 15 | "proposal", 16 | "[proposal]", 17 | "feature", 18 | "feature request", 19 | "[feature]", 20 | "[feature request]", 21 | ":" 22 | }; 23 | 24 | foreach (var prefix in prefixes) 25 | { 26 | if (title.StartsWith(prefix, StringComparison.OrdinalIgnoreCase)) 27 | title = title.Substring(prefix.Length).Trim(); 28 | } 29 | 30 | return title; 31 | } 32 | 33 | public static string FixLabel(string labelName) 34 | { 35 | while (true) 36 | { 37 | var firstColon = labelName.IndexOf(':'); 38 | if (firstColon < 0) 39 | break; 40 | 41 | var secondColon = labelName.IndexOf(':', firstColon + 1); 42 | if (secondColon < 0) 43 | break; 44 | 45 | var emojiStart = firstColon; 46 | var emojiLength = secondColon - emojiStart + 1; 47 | labelName = labelName.Remove(emojiStart, emojiLength); 48 | } 49 | 50 | return labelName.Trim(); 51 | } 52 | 53 | public static string GetMarkdownLink(string owner, string repo, int id, string url, string title) 54 | { 55 | var fixedTitle = FixTitle(title); 56 | return $"[{owner}/{repo}#{id}: {fixedTitle}]({url})"; 57 | } 58 | 59 | public static string GetHtmlLink(string owner, string repo, int id, string url, string title) 60 | { 61 | var fixedTitle = FixTitle(title); 62 | return $"{owner}/{repo}#{id}: {fixedTitle}"; 63 | } 64 | } 65 | } 66 | -------------------------------------------------------------------------------- /src/MainWindow.xaml: -------------------------------------------------------------------------------- 1 |  13 | 14 | 15 | 16 | 17 | 18 | 19 | 20 | 21 | 22 | 23 | 24 | 25 | 26 | 27 | 28 | 29 | 30 | 31 | 32 | 33 | 73 | 75 |