-
Notifications
You must be signed in to change notification settings - Fork 565
Open
Labels
gitlabIssue related to Scorecard's GitLab clientIssue related to Scorecard's GitLab clientgood first issueGood for newcomersGood for newcomerskind/bugSomething isn't workingSomething isn't working
Description
Describe the bug
Using the first commit returned has proven to not be the best source for the retrieval of merge requests details.
Expected behavior
The before field is intended to help limit the number of merge request details based on the last commit date for the default branch. By using the first record, this may not always include all of the related merge requests, throwing the scoring calculation off.
Additional context
I believe it would be better to calculate the date by cycling through the commits to determine the best date to use. I have a this is a working branch that I'm working through while validating the Code-Review
check's scores
Metadata
Metadata
Assignees
Labels
gitlabIssue related to Scorecard's GitLab clientIssue related to Scorecard's GitLab clientgood first issueGood for newcomersGood for newcomerskind/bugSomething isn't workingSomething isn't working
Type
Projects
Status
Backlog - Bugs