Several issues with `tap sdk gitversion`

  • tap sdk gitversion --log <n> crashes on Linux
  • tap sdk gitversion --log <n> outputs <n+2> lines and does not reset the console foreground color to the default value
  • no range/validity check on --log and --fields options
  • extend tap sdk gitversion ref so that ref is taken as a reference name (and not just a commit SHA-1 hash) as it is done for tap sdk gitversion --log <n> ref
  • no error handling for --log when ref is not contained in the git repository
  • tap sdk gitversion 2f0cc71bc0f2541ec48fc2701926c3a26c7dcd77 should be labeled
    9.8.0-alpha.34.1+2f0cc71b.249-SweepInputBug instead of
    9.8.0-alpha.33.3+2f0cc71b.249-SweepInputBug, since
    2f0cc71bc0f2541ec48fc2701926c3a26c7dcd77 is a parent of 582d1bdda07b059c1ee832f12a7b7aca4cc4260d that is labeled
    9.8.0-beta.34+582d1bdd
  • The output of tap sdk gitversion -c 0d2baa20 --log 10 should be
* │ │ │   9.8.2-rc.1+5ddea848 - Merge branch 'master' into 'release9x'
├─┤ │ │   
│ ├─┘ │   
│ *   │   9.8.2-beta.3+37c7478e - Merge branch '257-DownloadResume' into 'master'

instead of

* │ │ │   9.8.2-rc.1+5ddea848 - Merge branch 'master' into 'release9x'
├─┤ 
│ ├─┘ │   
│ *   │   9.8.2-beta.3+37c7478e - Merge branch '257-DownloadResume' into 'master'
  • The output of tap sdk gitversion 29228d4e --log 15 should be:
*     9.8.1-beta.1+29228d4e - Merge branch 'version_9_8_1' into 'master'
├─┐
│ *   9.8.1-alpha.1.1+802aa876.version-9-8-1 - Bump version 9.8.1
* │   9.8.0-beta.39+fff5cf25 - Merge branch '9.8.1.releasenote' into 'master'
├─│─┐
│ │ * 9.8.0-alpha.37.1+9c785c03.9-8-1-releasenote - Added bugfix #251 to release note
│ ├─┘ 
* │   9.8.0-beta.38+5f06ad14 - Merge branch '251-AvailableValuesInTestPlanReferenceAndParameters' into 'master'
├─┘   
├─┐
│ *   9.8.0-alpha.37.2+f8ba3f96.251-AvailableValuesInTestPlanR - Fixed TestHttprepositoryManagerRedirect

instead of:

*     9.8.1-beta.1+29228d4e - Merge branch 'version_9_8_1' into 'master'
├─┐
│ *   9.8.1-alpha.1.1+802aa876.version-9-8-1 - Bump version 9.8.1
* │   9.8.0-beta.39+fff5cf25 - Merge branch '9.8.1.releasenote' into 'master'
├─│─┐
│ │ * 9.8.0-alpha.37.1+9c785c03.9-8-1-releasenote - Added bugfix #251 to release note
│ ├─┘ 
* │   9.8.0-beta.38+5f06ad14 - Merge branch '251-AvailableValuesInTestPlanReferenceAndParameters' into 'master'
├───┐
│   * 9.8.0-alpha.37.2+f8ba3f96.251-AvailableValuesInTestPlanR - Fixed TestHttprepositoryManagerRedirect
Edited by Vivienne Spence