git-quick-stats icon indicating copy to clipboard operation
git-quick-stats copied to clipboard

Address inconsistency in changelogs

Open tomice opened this issue 1 year ago • 1 comments

  • Original function lacked explicit sorting by time. Adding sorting by time within the same day to help make changelogs consistent.
  • Removed format_date() and brought its functionality back into changelogs() as it wasn't used anywhere else

Fixes #161

tomice avatar Oct 12 '24 21:10 tomice

Tested with the provided repo. We now get the following without an author:

ice@3900x:Shapes-Dataset(main)$ ./git-quick-stats -c
Git changelogs:


[2022-03-24]
 * Update README.md (Daraan)

[2022-03-10]
 * Update README.md (Daraan)
 * Update README.md (Daraan)
 * Added Classifier and images (Daraan)
 * Added generator and updated help (Daraan)

[2022-03-09]
 * Initial commit (Daraan)

And the following with an author:

ice@3900x:Shapes-Dataset(main)$ ./git-quick-stats -L
Which author? Daraan
Git changelogs for author 'Daraan':


[2022-03-24]
* Update README.md (Daraan)

[2022-03-10]
* Update README.md (Daraan)
* Update README.md (Daraan)
* Added Classifier and images (Daraan)
* Added generator and updated help (Daraan)

[2022-03-09]
* Initial commit (Daraan)
ice@3900x:Shapes-Dataset(main)$

Also tested on our own repo:

ice@3900x:git-quick-stats(bugfix/161-changelogs-wrong-output)$ git log -n 13 --no-merges --oneline
fb488ab (HEAD -> bugfix/161-changelogs-wrong-output, origin/bugfix/161-changelogs-wrong-output) Address inconsistency in changelogs
d4e56cf Adjust commitsByYear() to support multiple dates
cf239e5 (tag: 2.5.7) Update README.md
7d50877 Address more feedback
fe2df31 Address review feedback
6155634 Fix stats for situation where there are no inserts / deletes
8c1d81a Add a list of new contributors
5b01b5e Update documentation for clarity
cd6d67d improve macos error message wrt gnu date
b525ed3 Create new repo if running tests in non-git area
41a8542 Handle error where BSD date is being used
43bc825 calculate the day before the given since date as it excludes the given daten the inclusion is wanted
62b128a docs: Document default `_GIT_LIMIT` value
ice@3900x:git-quick-stats(bugfix/161-changelogs-wrong-output)$ ./git-quick-stats -c
Git changelogs:


[2024-10-12]
 * Address inconsistency in changelogs (Tom Ice)

[2024-10-01]
 * Adjust commitsByYear() to support multiple dates (Tom Ice)

[2024-09-19]
 * Update README.md (Tom Ice)
 * Address more feedback (Ben Cotton)
 * Address review feedback (Ben Cotton)

[2024-09-17]
 * Fix stats for situation where there are no inserts / deletes (Michael Robinson)

[2024-06-26]
 * Add a list of new contributors (Ben Cotton)

[2024-05-16]
 * Update documentation for clarity (Tom Ice)

[2024-04-28]
 * improve macos error message wrt gnu date (Rui Chen)

[2024-04-20]
 * Create new repo if running tests in non-git area (Tom Ice)
 * Handle error where BSD date is being used (Tom Ice)

[2024-01-09]
 * calculate the day before the given since date as it excludes the given daten the inclusion is wanted (Martin Schaaf)

[2023-11-18]
 * docs: Document default `_GIT_LIMIT` value (Edwin Kofler)
ice@3900x:git-quick-stats(bugfix/161-changelogs-wrong-output)$

tomice avatar Oct 12 '24 21:10 tomice