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 an option to not have full depth when no tag matches the expression and config to limit the depth #336

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Commits on Sep 21, 2024

  1. Implement distanceOrZero to have 0 instead of depth when tag is not f…

    …ound
    Marc ROZANC authored and mrozanc committed Sep 21, 2024
    Configuration menu
    Copy the full SHA
    4c1cfe5 View commit details
    Browse the repository at this point in the history

Commits on Sep 22, 2024

  1. Update documentation

    mrozanc committed Sep 22, 2024
    Configuration menu
    Copy the full SHA
    e9f82b4 View commit details
    Browse the repository at this point in the history
  2. Rename boolean for foundTag and add getter

    Marc ROZANC committed Sep 22, 2024
    Configuration menu
    Copy the full SHA
    b094b98 View commit details
    Browse the repository at this point in the history
  3. Add describeTagMaxDepth config to limit the repository scan

    In case we have a more specific tag pattern to search on a branch,
     and this pattern does not match any tag in the branch, it's preferable
     to have a limit to the describe search for performance reasons.
    Marc ROZANC committed Sep 22, 2024
    Configuration menu
    Copy the full SHA
    b9bd821 View commit details
    Browse the repository at this point in the history
  4. Fix distance implementation with multiple parents

    As the implementation was a counter, when visiting many parents without resetting the counter,
    the returned distance was not correct.
    
    Using the DepthWalk.RevWalk has two benefits:
     - it can limit the depths
     - it allows accessing the depth of the current commit instead of trying to do it with custom code
    Marc ROZANC committed Sep 22, 2024
    Configuration menu
    Copy the full SHA
    4731084 View commit details
    Browse the repository at this point in the history
  5. Integration test fixes

     - Tags at same depth: no logic implemented to specify which one is selected, so I put one
      more commit in the test to have different depths.
     - Version from label=0 plus distance=0 (single commit, no tag), not sure why the test was expecting "1" as a version.
    Marc ROZANC committed Sep 22, 2024
    Configuration menu
    Copy the full SHA
    92618f7 View commit details
    Browse the repository at this point in the history

Commits on Oct 4, 2024

  1. Update README.md

    mrozanc committed Oct 4, 2024
    Configuration menu
    Copy the full SHA
    180bcb4 View commit details
    Browse the repository at this point in the history

Commits on Oct 17, 2024

  1. Remove wildcard import

    mrozanc committed Oct 17, 2024
    Configuration menu
    Copy the full SHA
    51255ce View commit details
    Browse the repository at this point in the history