Skip to content
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

Add commit url output option #9

Merged
merged 1 commit into from
Sep 9, 2024
Merged

Add commit url output option #9

merged 1 commit into from
Sep 9, 2024

Conversation

clamey
Copy link
Contributor

@clamey clamey commented Sep 9, 2024

Adds an optional boolean flag called --commit-urls or --no-commit-urls for outputting commit URLs for easier commit validation, which defaults to off to keep backwards compatibility.

Without the new flag, it's the same output as before this change:

topher> pipenv run python3 github-repo-committers.py --access_token $ACCESS_TOKEN --repo_name "kaakaww/contributors_tool"                                                                         <region:us-west-2>
In the repository 'kaakaww/contributors_tool', there are 4 contributor(s) over 90 days with the earliest commit on 2021-08-04 01:05:47.
Here is the list of Github contributors:
sgerlach: 2021-11-02T01:05:47Z
jeremygoldsmith: 2021-10-31T04:07:22Z
azconger: 2023-03-22T23:23:11Z
clamey: 2024-09-09T17:31:05Z

With the new flag, we get a URL for each commit:

topher> pipenv run python3 github-repo-committers.py --access_token $ACCESS_TOKEN --repo_name "kaakaww/contributors_tool" --commit_urls                                                             <region:us-west-2>
In the repository 'kaakaww/contributors_tool', there are 4 contributor(s) over 90 days with the earliest commit on 2021-08-04 01:05:47.
Here is the list of Github contributors:
sgerlach: 2021-11-02T01:05:47Z, https://github.com/kaakaww/contributors_tool/commit/66eee5bb5b32d93562ccf6891a24bf747c621a10
jeremygoldsmith: 2021-10-31T04:07:22Z, https://github.com/kaakaww/contributors_tool/commit/48fa39f9b056cff16c7e3cb39bf598ce3914fc79
azconger: 2023-03-22T23:23:11Z, https://github.com/kaakaww/contributors_tool/commit/c9e8b1a7c52705075d35654a863ae96418f11ea0
clamey: 2024-09-09T17:31:05Z, https://github.com/kaakaww/contributors_tool/commit/cb667e6422ea0acea2dcfa15c4550adf85856793

@clamey clamey merged commit 3c72aa1 into main Sep 9, 2024
3 checks passed
@clamey clamey deleted the output-commit-urls branch September 9, 2024 21:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants