-
Notifications
You must be signed in to change notification settings - Fork 15
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
fix: re-run #320
fix: re-run #320
Conversation
✅ Deploy Preview for reviewbot-x canceled.
|
server.go
Outdated
installationID := event.GetInstallation().GetID() | ||
plist, err := linters.FilterPullRequestsWithCommit(ctx, s.GithubClient(installationID), org, repo, headSha) | ||
if err != nil { | ||
log.Debugf("Filter pullreqeust fail %v\n", err) |
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.
为何是Debug级别?
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.
已经调整
882f1e9
to
4472a87
Compare
internal/linters/github.go
Outdated
StartedAt: &startTime, | ||
Name: linterName, | ||
HeadSHA: headSha, | ||
DetailsURL: &url, |
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.
这是误加?
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #320 +/- ##
==========================================
- Coverage 37.22% 36.52% -0.70%
==========================================
Files 27 27
Lines 1996 2034 +38
==========================================
Hits 743 743
- Misses 1158 1196 +38
Partials 95 95 ☔ View full report in Codecov by Sentry. |
7b3af55
to
c87d3b3
Compare
internal/linters/github.go
Outdated
repullRequests = append(repullRequests, pullRequest) | ||
} | ||
} | ||
if err != nil { |
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.
要先处理err
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.
done
internal/linters/github.go
Outdated
func FilterPullRequestsWithCommit(ctx context.Context, gc *github.Client, owner string, repo string, headSha string) ([]*github.PullRequest, error) { | ||
plopt := github.PullRequestListOptions{} | ||
var repullRequests []*github.PullRequest | ||
pullRequests, _, err := gc.PullRequests.List(ctx, owner, repo, &plopt) |
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.
response 应该也要处理
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.
加了response的处理
No description provided.