Double-check that commits(first: 100 returns the most recent 100 commits, and not the oldest 100

[mmarchini]

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 :)

[targos]

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: 5 to get the most recent commits (still ordered from old to new):

[codebytere]

Meaning we should swap to last?

[mmarchini]

Meaning we should swap to last?

That's what I understood, yes

[github-actions]

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.