Skip to content

fix(ci): avoid failing release job when tag already exists#145

Merged
JerrettDavis merged 1 commit intomainfrom
fix/ci-tag-exists-check
Mar 27, 2026
Merged

fix(ci): avoid failing release job when tag already exists#145
JerrettDavis merged 1 commit intomainfrom
fix/ci-tag-exists-check

Conversation

@JerrettDavis
Copy link
Copy Markdown
Owner

Problem\nRelease job in CI was failing on main when � already existed remotely.\n\nThe workflow used git rev-parse (local check), which can miss remote-only tags and still attempt git push origin vX, causing a hard failure.\n\n## Fix\n- switch tag existence check to remote source of truth: git ls-remote --tags origin refs/tags/\n- skip tag push when tag exists on origin\n\n## Why this helps\nThis prevents the release workflow from failing just because the tag already exists, keeping CI/CD green and idempotent for repeated runs on the same package version.

@chatgpt-codex-connector
Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@github-actions
Copy link
Copy Markdown
Contributor

⚠️ Deprecation Warning: The deny-licenses option is deprecated for possible removal in the next major release. For more information, see issue 997.

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Snapshot Warnings

⚠️: No snapshots were found for the head SHA 776c658.
Ensure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice.

Scanned Files

None

@github-actions
Copy link
Copy Markdown
Contributor

Test Results

417 tests   417 ✅  1m 29s ⏱️
  1 suites    0 💤
  1 files      0 ❌

Results for commit 776c658.

@github-actions
Copy link
Copy Markdown
Contributor

🔍 PR Validation Results

Version: ``

✅ Validation Steps

  • Build solution
  • Run tests
  • Build documentation
  • Dry-run NuGet packaging

📊 Artifacts

Dry-run artifacts have been uploaded and will be available for 7 days.


This comment was automatically generated by the PR validation workflow.

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 27, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.18%. Comparing base (f1d84b3) to head (776c658).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #145      +/-   ##
==========================================
+ Coverage   81.14%   81.18%   +0.03%     
==========================================
  Files         196      196              
  Lines       19134    19134              
  Branches     2712     2712              
==========================================
+ Hits        15527    15533       +6     
+ Misses       2880     2879       -1     
+ Partials      727      722       -5     
Flag Coverage Δ
unittests 81.18% <ø> (+0.03%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@JerrettDavis JerrettDavis merged commit 7951503 into main Mar 27, 2026
10 of 11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant