Skip to content

Conversation

@jkloetzke
Copy link
Member

@aaronpohle Could you test this MR and see if it fixes your problem?

jkloetzke added 3 commits July 1, 2023 23:32
Git does not list nested tags with "git tag --points-at=...". Reverse
the logic for the test and compare the HEAD and the tag commits. Only if
they differ try to get the list of tags that point to the current HEAD.

Fixes BobBuildTool#520.
@codecov
Copy link

codecov bot commented Jul 1, 2023

Codecov Report

Patch coverage: 100.00% and project coverage change: -0.01 ⚠️

Comparison is base (33fdae7) 88.18% compared to head (2c84a73) 88.17%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #524      +/-   ##
==========================================
- Coverage   88.18%   88.17%   -0.01%     
==========================================
  Files          46       46              
  Lines       14460    14458       -2     
==========================================
- Hits        12752    12749       -3     
- Misses       1708     1709       +1     
Impacted Files Coverage Δ
pym/bob/scm/git.py 93.63% <100.00%> (-0.20%) ⬇️

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@jkloetzke jkloetzke merged commit 2be1d30 into BobBuildTool:master Jul 5, 2023
@jkloetzke jkloetzke deleted the fix-520 branch July 5, 2023 18:53
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.

1 participant