Skip to content

Commit

Permalink
fix: Make sure that the tags are fetched before testing with git desc…
Browse files Browse the repository at this point in the history
…ribe (#2047)

## Description:
This fixes the issue seen in
https://app.circleci.com/pipelines/github/kurtosis-tech/kurtosis/10677/workflows/66565433-39e0-4397-b0d8-4c3c81ad366b/jobs/152621.

When ci checks out code in a step it doesn't fetch tags, so `git
describe --exact-match` will always fail - this change fixes this by
fetching tags.

## Is this change user facing?
NO

## References (if applicable):
-
https://app.circleci.com/pipelines/github/kurtosis-tech/kurtosis/10677/workflows/66565433-39e0-4397-b0d8-4c3c81ad366b/jobs/152621
-#2041
  • Loading branch information
Dartoxian committed Jan 16, 2024
1 parent 5d50625 commit b8e7afb
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions .circleci/config.yml
Expand Up @@ -435,6 +435,7 @@ jobs:

- run: |
# If the current commit is not a tag and if there is no change (compared to main) of anything but docs, then skip this
git fetch origin --tags
if (! git describe --exact-match) && git --no-pager diff --exit-code origin/main...HEAD -- . ':!docs' ':!*.md' ; then
circleci-agent step halt
fi
Expand Down

0 comments on commit b8e7afb

Please sign in to comment.