Skip to content

[Feature] Add stats and files in the response of repos().list_commits() (currently both are None) #767

Open
@klion26

Description

@klion26

Currently, the api repos().list_commits() will always return None for stats and files, debug it a bit I found that:

  1. repos().list_commits() will call repos/{owner}/{repo}/commits from api.github.com but the response of this API did not contain the stats and files information
  2. We need to call repos/{owner}/{repo}/commits/{sha} to get the stats and files information.

If this is valid, I can contribute after reaching consensus.

attached two example links[1][2] for more info, and the doc for commits[3]

[1] https://api.github.com/repos/apache/amoro/commits?author=klion26
[2] repos/apache/amoro/commits/c69a2363a6263297421ec6bece176b39a69a5c99
[3] https://docs.github.com/en/enterprise-cloud@latest/rest/commits/commits?apiVersion=2022-11-28

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions