-
Notifications
You must be signed in to change notification settings - Fork 76
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
Create a simple dashboard to view all pull requests by who should review them #774
base: main
Are you sure you want to change the base?
Create a simple dashboard to view all pull requests by who should review them #774
Conversation
src/GitHubCodeReviewDashboard/GitHubCodeReviewDashboard/Dashboard.cs
Outdated
Show resolved
Hide resolved
var github = new GitHubClient(new ProductHeaderValue("dotnet-roslyn-Code-Review-Dashboard")); | ||
github.Credentials = new Credentials(Startup.GitHubToken); | ||
var openPullRequests = await GetAllPullRequests(github); | ||
var ideTeamMembers = (await github.Organization.Team.GetAllMembers(1781706)).Select(u => u.Login).ToList(); |
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.
constant?
} | ||
|
||
pullRequests.Add(pullRequest); | ||
} |
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.
move to bottom
src/GitHubCodeReviewDashboard/GitHubCodeReviewDashboard/Dashboard.cs
Outdated
Show resolved
Hide resolved
src/GitHubCodeReviewDashboard/GitHubCodeReviewDashboard/GitHubCodeReviewDashboard.csproj
Outdated
Show resolved
Hide resolved
src/GitHubCodeReviewDashboard/GitHubCodeReviewDashboard/Pages/Index.cshtml
Outdated
Show resolved
Hide resolved
|
||
public void OnGet() | ||
{ | ||
} |
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.
?
--> | ||
<Project ToolsVersion="4.0" xmlns="http://schemas.microsoft.com/developer/msbuild/2003"> | ||
<PropertyGroup> | ||
<WebPublishMethod>MSDeploy</WebPublishMethod> |
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.
what is this for?
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.
Let's you deploy straight to Azure from the UI. This file is mostly autogenerated.
<Project ToolsVersion="4.0" xmlns="http://schemas.microsoft.com/developer/msbuild/2003"> | ||
<PropertyGroup> | ||
<WebPublishMethod>MSDeploy</WebPublishMethod> | ||
<IsLinux>true</IsLinux> |
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.
:'(
...iewDashboard/GitHubCodeReviewDashboard/Properties/PublishProfiles/Web Deploy to Azure.pubxml
Outdated
Show resolved
Hide resolved
@@ -0,0 +1,46 @@ | |||
using Microsoft.AspNetCore.Builder; |
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.
copyrights?
"Microsoft.Hosting.Lifetime": "Information" | ||
} | ||
}, | ||
"AllowedHosts": "*" |
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.
?
src/GitHubCodeReviewDashboard/GitHubCodeReviewDashboard/wwwroot/css/site.css
Show resolved
Hide resolved
src/GitHubCodeReviewDashboard/GitHubCodeReviewDashboard/wwwroot/js/site.js
Show resolved
Hide resolved
15824d8
to
20f631e
Compare
@@ -0,0 +1,81 @@ | |||
using Octokit; |
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.
Do we need license headers for this repo?
@jasonmalinowski Do we want to move forward with this PR? |
@JoeRobich Hmm, yeah, I should merge this. I had deployed it to Azure as a test, which I need to fix up after some subscription changes. |
2b2e63f
to
c0ab9d4
Compare
This is the code powering https://roslyncodereviewdashboard.azurewebsites.net/.