Repository navigation
Double-check that commits(first: 100 returns the most recent 100 commits, and not the oldest 100 #477
Description
As title says, our PRCommits.gql query uses first for commits. I'm not sure if it fetches the newest or oldest 100 commits. If whoever implemented this already tested it and remember the answer, that's enough for me too :)
Activity
targos commentedon Aug 15, 2020
I just tested with https://developer.github.com/v4/explorer for that PR: nodejs/node#34752
With
first: 5, it returns them from oldest to newest:You can change to
last: 5to get the most recent commits (still ordered from old to new):codebytere commentedon Aug 17, 2020
Meaning we should swap to
last?mmarchini commentedon Aug 17, 2020
That's what I understood, yes
github-actions commentedon Nov 16, 2020
This issue is stale because it has been open many days with no activity. It will be closed soon unless the stale label is removed or a comment is made.