Fix tag handling: preserve annotations and explicit fetch-tags #2356

ericsciple · 2026-01-08T20:01:27Z

Summary

Fixes multiple issues with tag handling in the checkout action.

Issues Fixed

1. `fetch-tags: true` doesn't fetch tags (#1471)

Previously, when using fetch-tags: true on a branch checkout, tags were not actually fetched. Now tags are properly included in the fetch refspec.

2. Annotated tags converted to lightweight (#290)

When checking out an annotated tag, the tag annotation was lost because the action fetched by commit hash rather than by tag reference. Now tags are fetched via explicit refspec (+refs/tags/*:refs/tags/*), which fetches the actual tag objects and preserves annotations.

3. Fatal error with tag + `fetch-tags: true` (#1467)

Checking out a tag with fetch-tags: true caused a fatal error due to duplicate refspecs. Now only the wildcard refspec is used (the specific tag is redundant when fetching all tags).

Testing

Unit Tests

Added new unit tests.

E2E Tests

Scenario	Result
Tag + fetch-tags: true	✅ Pass
Tag + fetch-tags: false	✅ Pass
Branch + fetch-tags: true	✅ Pass
Branch + fetch-tags: false	✅ Pass
Shallow clone + tags	✅ Pass
Full clone + tags	✅ Pass
Cross-platform (Ubuntu, Windows, macOS)	✅ Pass
Annotated tag moved after trigger	✅ Fails with clear error
Lightweight tag moved after trigger	✅ Fails with clear error

src/ref-helper.ts

src/git-source-provider.ts

Copilot

Pull request overview

This pull request fixes multiple tag handling issues in the checkout action by changing how tags are fetched and verified.

Tags are now fetched by reference (not commit hash) to preserve annotations
The fetch-tags option now properly controls tag fetching via explicit refspecs
Added validation to detect when tags have been moved after workflow trigger (for shallow fetches)

Reviewed changes

Copilot reviewed 7 out of 8 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
src/ref-helper.ts	Modified `getRefSpec` to accept `fetchTags` parameter and fetch tags by ref instead of commit; updated `testRef` to use `^{commit}` for annotated tag dereferencing
src/git-source-provider.ts	Removed `fetchTags` from fetch options (now in refspec); added tag movement validation for shallow fetches
src/git-command-manager.ts	Removed `fetchTags` option from fetch method; always use `--no-tags` with explicit tag control via refspecs
test/ref-helper.test.ts	Added comprehensive tests for new `fetchTags` parameter behavior
test/git-command-manager.test.ts	Updated tests to reflect new fetch behavior with `--no-tags` always present
test/verify-fetch-tags.sh	New script to verify tags are properly fetched in E2E tests
.github/workflows/test.yml	Added E2E test for `fetch-tags: true` functionality
dist/index.js	Compiled changes matching source modifications

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

src/git-source-provider.ts

This PR fixes several issues with tag handling in the checkout action: 1. fetch-tags: true now works (fixes #1471) - Tags refspec is now included in getRefSpec() when fetchTags=true - Previously tags were only fetched during a separate fetch that was overwritten by the main fetch 2. Tag checkout preserves annotations (fixes #290) - Tags are fetched via refspec (+refs/tags/*:refs/tags/*) instead of --tags flag - This fetches the actual tag objects, preserving annotations 3. Tag checkout with fetch-tags: true no longer fails (fixes #1467) - When checking out a tag with fetchTags=true, only the wildcard refspec is used (specific tag refspec is redundant) Changes: - src/ref-helper.ts: getRefSpec() now accepts fetchTags parameter and prepends tags refspec when true - src/git-command-manager.ts: fetch() simplified to always use --no-tags, tags are fetched explicitly via refspec - src/git-source-provider.ts: passes fetchTags to getRefSpec() - Added E2E test for fetch-tags option Related #1471, #1467, #290

src/ref-helper.ts

ben-manes · 2026-01-11T20:59:51Z

@ericsciple This is fixed in v6.0.2 but that is marked as a pre-release. Was that a mistake and, if not, why not mark it as -beta? This is causing action updates tools to skip/flag it, so I wanted to make sure you were aware of that release marker.

ericsciple · 2026-01-22T16:46:09Z

@ben-manes thanks! I just now removed the pre-release flag (I had originally planned to remove it sooner).

I will watch for issues, then bump the v6 tag to match next week.

This PR contains the following updates: | Package | Type | Update | Change | |---|---|---|---| | [actions/checkout](https://github.com/actions/checkout) | action | patch | `v6.0.1` → `v6.0.2` | --- ### Release Notes <details> <summary>actions/checkout (actions/checkout)</summary> ### [`v6.0.2`](https://github.com/actions/checkout/blob/HEAD/CHANGELOG.md#v602) [Compare Source](actions/checkout@v6.0.1...v6.0.2) - Fix tag handling: preserve annotations and explicit fetch-tags by [@ericsciple](https://github.com/ericsciple) in [#2356](actions/checkout#2356) </details> --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR is behind base branch, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, check this box --- This PR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).  Reviewed-on: https://git.tainton.uk/luke/instant-msg-api/pulls/238 Reviewed-by: Luke Tainton <[email protected]> Co-authored-by: renovate[bot] <[email protected]> Co-committed-by: renovate[bot] <[email protected]>

This PR contains the following updates: | Package | Type | Update | Change | |---|---|---|---| | [actions/checkout](https://github.com/actions/checkout) | action | patch | `v6.0.1` → `v6.0.2` | --- ### Release Notes <details> <summary>actions/checkout (actions/checkout)</summary> ### [`v6.0.2`](https://github.com/actions/checkout/blob/HEAD/CHANGELOG.md#v602) [Compare Source](actions/checkout@v6.0.1...v6.0.2) - Fix tag handling: preserve annotations and explicit fetch-tags by [@ericsciple](https://github.com/ericsciple) in [#2356](actions/checkout#2356) </details> --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR is behind base branch, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, check this box --- This PR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).  Reviewed-on: https://git.tainton.uk/actions/gha-workflows/pulls/58 Co-authored-by: renovate[bot] <[email protected]> Co-committed-by: renovate[bot] <[email protected]>

@ericsciple

| datasource | package | from | to | | ----------- | ---------------- | ------ | ------ | | github-tags | actions/checkout | v4.1.7 | v6.0.2 | ## [vv6.0.2](https://github.com/actions/checkout/blob/HEAD/CHANGELOG.md#v602) - Fix tag handling: preserve annotations and explicit fetch-tags by [@ericsciple](https://github.com/ericsciple) in [#2356](actions/checkout#2356) ## [vv6.0.1](actions/checkout@v6.0.0...v6.0.1) ## [vv6.0.0](actions/checkout@v5.0.1...v6.0.0) ## [vv5.0.1](https://github.com/actions/checkout/releases/tag/v5.0.1) ##### What's Changed - Port v6 cleanup to v5 by [@ericsciple](https://github.com/ericsciple) in [#2301](actions/checkout#2301) **Full Changelog**: <actions/checkout@v5...v5.0.1> ## [vv5.0.0](https://github.com/actions/checkout/releases/tag/v5.0.0) ##### What's Changed - Update actions checkout to use node 24 by [@salmanmkc](https://github.com/salmanmkc) in [#2226](actions/checkout#2226) - Prepare v5.0.0 release by [@salmanmkc](https://github.com/salmanmkc) in [#2238](actions/checkout#2238) ##### ⚠️ Minimum Compatible Runner Version **v2.327.1**\ [Release Notes](https://github.com/actions/runner/releases/tag/v2.327.1) Make sure your runner is updated to this version or newer to use this release. **Full Changelog**: <actions/checkout@v4...v5.0.0> ## [vv4.3.1](https://github.com/actions/checkout/releases/tag/v4.3.1) ##### What's Changed - Port v6 cleanup to v4 by [@ericsciple](https://github.com/ericsciple) in [#2305](actions/checkout#2305) **Full Changelog**: <actions/checkout@v4...v4.3.1> ## [vv4.3.0](https://github.com/actions/checkout/releases/tag/v4.3.0) ##### What's Changed - docs: update README.md by [@motss](https://github.com/motss) in [#1971](actions/checkout#1971) - Add internal repos for checking out multiple repositories by [@mouismail](https://github.com/mouismail) in [#1977](actions/checkout#1977) - Documentation update - add recommended permissions to Readme by [@benwells](https://github.com/benwells) in [#2043](actions/checkout#2043) - Adjust positioning of user email note and permissions heading by [@joshmgross](https://github.com/joshmgross) in [#2044](actions/checkout#2044) - Update README.md by [@nebuk89](https://github.com/nebuk89) in [#2194](actions/checkout#2194) - Update CODEOWNERS for actions by [@TingluoHuang](https://github.com/TingluoHuang) in [#2224](actions/checkout#2224) - Update package dependencies by [@salmanmkc](https://github.com/salmanmkc) in [#2236](actions/checkout#2236) - Prepare release v4.3.0 by [@salmanmkc](https://github.com/salmanmkc) in [#2237](actions/checkout#2237) ##### New Contributors - [@motss](https://github.com/motss) made their first contribution in [#1971](actions/checkout#1971) - [@mouismail](https://github.com/mouismail) made their first contribution in [#1977](actions/checkout#1977) - [@benwells](https://github.com/benwells) made their first contribution in [#2043](actions/checkout#2043) - [@nebuk89](https://github.com/nebuk89) made their first contribution in [#2194](actions/checkout#2194) - [@salmanmkc](https://github.com/salmanmkc) made their first contribution in [#2236](actions/checkout#2236) **Full Changelog**: <actions/checkout@v4...v4.3.0> ## [vv4.2.2](https://github.com/actions/checkout/blob/HEAD/CHANGELOG.md#v422) - `url-helper.ts` now leverages well-known environment variables by [@jww3](https://github.com/jww3) in [#1941](actions/checkout#1941) - Expand unit test coverage for `isGhes` by [@jww3](https://github.com/jww3) in [#1946](actions/checkout#1946) ## [vv4.2.1](https://github.com/actions/checkout/blob/HEAD/CHANGELOG.md#v421) - Check out other refs/\* by commit if provided, fall back to ref by [@orhantoy](https://github.com/orhantoy) in [#1924](actions/checkout#1924) ## [vv4.2.0](https://github.com/actions/checkout/blob/HEAD/CHANGELOG.md#v420) - Add Ref and Commit outputs by [@lucacome](https://github.com/lucacome) in [#1180](actions/checkout#1180) - Dependency updates by [@dependabot-](https://github.com/dependabot-) [#1777](actions/checkout#1777), [#1872](actions/checkout#1872)

@ericsciple

##### [\`v6.0.2\`](https://github.com/actions/checkout/blob/HEAD/CHANGELOG.md#v602) - Fix tag handling: preserve annotations and explicit fetch-tags by [@ericsciple](https://github.com/ericsciple) in [#2356](actions/checkout#2356)

Bumps [actions/checkout](https://github.com/actions/checkout) from 6.0.1 to 6.0.2. Release notes *Sourced from [actions/checkout's releases](https://github.com/actions/checkout/releases).* > v6.0.2 > ------ > > What's Changed > -------------- > > * Add orchestration\_id to git user-agent when ACTIONS\_ORCHESTRATION\_ID is set by [`@TingluoHuang`](https://github.com/TingluoHuang) in [actions/checkout#2355](https://redirect.github.com/actions/checkout/pull/2355) > * Fix tag handling: preserve annotations and explicit fetch-tags by [`@ericsciple`](https://github.com/ericsciple) in [actions/checkout#2356](https://redirect.github.com/actions/checkout/pull/2356) > > **Full Changelog**: <actions/checkout@v6.0.1...v6.0.2> Changelog *Sourced from [actions/checkout's changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md).* > Changelog > ========= > > v6.0.2 > ------ > > * Fix tag handling: preserve annotations and explicit fetch-tags by [`@ericsciple`](https://github.com/ericsciple) in [actions/checkout#2356](https://redirect.github.com/actions/checkout/pull/2356) > > v6.0.1 > ------ > > * Add worktree support for persist-credentials includeIf by [`@ericsciple`](https://github.com/ericsciple) in [actions/checkout#2327](https://redirect.github.com/actions/checkout/pull/2327) > > v6.0.0 > ------ > > * Persist creds to a separate file by [`@ericsciple`](https://github.com/ericsciple) in [actions/checkout#2286](https://redirect.github.com/actions/checkout/pull/2286) > * Update README to include Node.js 24 support details and requirements by [`@salmanmkc`](https://github.com/salmanmkc) in [actions/checkout#2248](https://redirect.github.com/actions/checkout/pull/2248) > > v5.0.1 > ------ > > * Port v6 cleanup to v5 by [`@ericsciple`](https://github.com/ericsciple) in [actions/checkout#2301](https://redirect.github.com/actions/checkout/pull/2301) > > v5.0.0 > ------ > > * Update actions checkout to use node 24 by [`@salmanmkc`](https://github.com/salmanmkc) in [actions/checkout#2226](https://redirect.github.com/actions/checkout/pull/2226) > > v4.3.1 > ------ > > * Port v6 cleanup to v4 by [`@ericsciple`](https://github.com/ericsciple) in [actions/checkout#2305](https://redirect.github.com/actions/checkout/pull/2305) > > v4.3.0 > ------ > > * docs: update README.md by [`@motss`](https://github.com/motss) in [actions/checkout#1971](https://redirect.github.com/actions/checkout/pull/1971) > * Add internal repos for checking out multiple repositories by [`@mouismail`](https://github.com/mouismail) in [actions/checkout#1977](https://redirect.github.com/actions/checkout/pull/1977) > * Documentation update - add recommended permissions to Readme by [`@benwells`](https://github.com/benwells) in [actions/checkout#2043](https://redirect.github.com/actions/checkout/pull/2043) > * Adjust positioning of user email note and permissions heading by [`@joshmgross`](https://github.com/joshmgross) in [actions/checkout#2044](https://redirect.github.com/actions/checkout/pull/2044) > * Update README.md by [`@nebuk89`](https://github.com/nebuk89) in [actions/checkout#2194](https://redirect.github.com/actions/checkout/pull/2194) > * Update CODEOWNERS for actions by [`@TingluoHuang`](https://github.com/TingluoHuang) in [actions/checkout#2224](https://redirect.github.com/actions/checkout/pull/2224) > * Update package dependencies by [`@salmanmkc`](https://github.com/salmanmkc) in [actions/checkout#2236](https://redirect.github.com/actions/checkout/pull/2236) > > v4.2.2 > ------ > > * `url-helper.ts` now leverages well-known environment variables by [`@jww3`](https://github.com/jww3) in [actions/checkout#1941](https://redirect.github.com/actions/checkout/pull/1941) > * Expand unit test coverage for `isGhes` by [`@jww3`](https://github.com/jww3) in [actions/checkout#1946](https://redirect.github.com/actions/checkout/pull/1946) > > v4.2.1 > ------ > > * Check out other refs/\* by commit if provided, fall back to ref by [`@orhantoy`](https://github.com/orhantoy) in [actions/checkout#1924](https://redirect.github.com/actions/checkout/pull/1924) > > v4.2.0 > ------ > > * Add Ref and Commit outputs by [`@lucacome`](https://github.com/lucacome) in [actions/checkout#1180](https://redirect.github.com/actions/checkout/pull/1180) > * Dependency updates by [`@dependabot`](https://github.com/dependabot)- [actions/checkout#1777](https://redirect.github.com/actions/checkout/pull/1777), [actions/checkout#1872](https://redirect.github.com/actions/checkout/pull/1872) > > v4.1.7 > ------ > > * Bump the minor-npm-dependencies group across 1 directory with 4 updates by [`@dependabot`](https://github.com/dependabot) in [actions/checkout#1739](https://redirect.github.com/actions/checkout/pull/1739) > * Bump actions/checkout from 3 to 4 by [`@dependabot`](https://github.com/dependabot) in [actions/checkout#1697](https://redirect.github.com/actions/checkout/pull/1697) > * Check out other refs/\* by commit by [`@orhantoy`](https://github.com/orhantoy) in [actions/checkout#1774](https://redirect.github.com/actions/checkout/pull/1774) > * Pin actions/checkout's own workflows to a known, good, stable version. by [`@jww3`](https://github.com/jww3) in [actions/checkout#1776](https://redirect.github.com/actions/checkout/pull/1776) > > v4.1.6 > ------ > > * Check platform to set archive extension appropriately by [`@cory-miller`](https://github.com/cory-miller) in [actions/checkout#1732](https://redirect.github.com/actions/checkout/pull/1732) ... (truncated) Commits * [`de0fac2`](actions/checkout@de0fac2) Fix tag handling: preserve annotations and explicit fetch-tags ([#2356](https://redirect.github.com/actions/checkout/issues/2356)) * [`064fe7f`](actions/checkout@064fe7f) Add orchestration\_id to git user-agent when ACTIONS\_ORCHESTRATION\_ID is set (... * See full diff in [compare view](actions/checkout@8e8c483...de0fac2) [![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility\_score?dependency-name=actions/checkout&package-manager=github\_actions&previous-version=6.0.1&new-version=6.0.2)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores) Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`. [//]: # (dependabot-automerge-start) [//]: # (dependabot-automerge-end) --- Dependabot commands and options You can trigger Dependabot actions by commenting on this PR: - `@dependabot rebase` will rebase this PR - `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it - `@dependabot merge` will merge this PR after your CI passes on it - `@dependabot squash and merge` will squash and merge this PR after your CI passes on it - `@dependabot cancel merge` will cancel a previously requested merge and block automerging - `@dependabot reopen` will reopen this PR if it is closed - `@dependabot close` will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually - `@dependabot show ignore conditions` will show all of the ignore conditions of the specified dependency - `@dependabot ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Bumps actions/checkout from 5.0.0 to 6.0.2. ## Release notes Sourced from actions/checkout's releases. v6.0.2 What's Changed Add orchestration_id to git user-agent when ACTIONS_ORCHESTRATION_ID is set by @TingluoHuang in actions/checkout#2355 Fix tag handling: preserve annotations and explicit fetch-tags by @ericsciple in actions/checkout#2356 Full Changelog: actions/checkout@v6.0.1...v6.0.2 v6.0.1 What's Changed Update all references from v5 and v4 to v6 by @ericsciple in actions/checkout#2314 Add worktree support for persist-credentials includeIf by @ericsciple in actions/checkout#2327 Clarify v6 README by @ericsciple in actions/checkout#2328 Full Changelog: actions/checkout@v6...v6.0.1 v6.0.0 What's Changed Update README to include Node.js 24 support details and requirements by @salmanmkc in actions/checkout#2248 Persist creds to a separate file by @ericsciple in actions/checkout#2286 v6-beta by @ericsciple in actions/checkout#2298 update readme/changelog for v6 by @ericsciple in actions/checkout#2311 Full Changelog: actions/checkout@v5.0.0...v6.0.0 v6-beta What's Changed Updated persist-credentials to store the credentials under $RUNNER_TEMP instead of directly in the local git config. This requires a minimum Actions Runner version of v2.329.0 to access the persisted credentials for Docker container action scenarios. v5.0.1 What's Changed Port v6 cleanup to v5 by @ericsciple in actions/checkout#2301 Full Changelog: actions/checkout@v5...v5.0.1 ## Changelog Sourced from actions/checkout's changelog. Changelog v6.0.2 Fix tag handling: preserve annotations and explicit fetch-tags by @ericsciple in actions/checkout#2356 v6.0.1 Add worktree support for persist-credentials includeIf by @ericsciple in actions/checkout#2327 v6.0.0 Persist creds to a separate file by @ericsciple in actions/checkout#2286 Update README to include Node.js 24 support details and requirements by @salmanmkc in actions/checkout#2248 v5.0.1 Port v6 cleanup to v5 by @ericsciple in actions/checkout#2301 v5.0.0 Update actions checkout to use node 24 by @salmanmkc in actions/checkout#2226 v4.3.1 Port v6 cleanup to v4 by @ericsciple in actions/checkout#2305 v4.3.0 docs: update README.md by @motss in actions/checkout#1971 Add internal repos for checking out multiple repositories by @mouismail in actions/checkout#1977 Documentation update - add recommended permissions to Readme by @benwells in actions/checkout#2043 Adjust positioning of user email note and permissions heading by @joshmgross in actions/checkout#2044 Update README.md by @nebuk89 in actions/checkout#2194 Update CODEOWNERS for actions by @TingluoHuang in actions/checkout#2224 Update package dependencies by @salmanmkc in actions/checkout#2236 v4.2.2 url-helper.ts now leverages well-known environment variables by @jww3 in actions/checkout#1941 Expand unit test coverage for isGhes by @jww3 in actions/checkout#1946 v4.2.1 Check out other refs/* by commit if provided, fall back to ref by @orhantoy in actions/checkout#1924 v4.2.0 Add Ref and Commit outputs by @lucacome in actions/checkout#1180 Dependency updates by @dependabot- actions/checkout#1777, actions/checkout#1872 v4.1.7 Bump the minor-npm-dependencies group across 1 directory with 4 updates by @dependabot in actions/checkout#1739 Bump actions/checkout from 3 to 4 by @dependabot in actions/checkout#1697 Check out other refs/* by commit by @orhantoy in actions/checkout#1774 Pin actions/checkout's own workflows to a known, good, stable version. by @jww3 in actions/checkout#1776 v4.1.6 Check platform to set archive extension appropriately by @cory-miller in actions/checkout#1732 ... (truncated) ## Commits de0fac2 Fix tag handling: preserve annotations and explicit fetch-tags (#2356) 064fe7f Add orchestration_id to git user-agent when ACTIONS_ORCHESTRATION_ID is set ( 8e8c483 Clarify v6 README (#2328) 033fa0d Add worktree support for persist-credentials includeIf (#2327) c2d88d3 Update all references from v5 and v4 to v6 (#2314) 1af3b93 update readme/changelog for v6 (#2311) 71cf226 v6-beta (#2298) 069c695 Persist creds to a separate file (#2286) ff7abcd Update README to include Node.js 24 support details and requirements (#2248) See full diff in compare view ![Dependabot compatibility score](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores) Issue-ID: CIMAN-33 Signed-off-by: dependabot[bot] <[email protected]> Change-Id: I2efc697cecfe2aa7bb222b00975003050af2e33d GitHub-PR: #69 GitHub-Hash: b97dabb355259666 Signed-off-by: onap.gh2gerrit <[email protected]>

ericsciple force-pushed the users/erisciple/26-01-tags branch 7 times, most recently from f12cf15 to a70240b Compare January 8, 2026 23:59

ericsciple commented Jan 9, 2026

View reviewed changes

src/ref-helper.ts Show resolved Hide resolved

ericsciple force-pushed the users/erisciple/26-01-tags branch from a70240b to 849c4d7 Compare January 9, 2026 00:26

ericsciple commented Jan 9, 2026

View reviewed changes

src/git-source-provider.ts Show resolved Hide resolved

ericsciple marked this pull request as ready for review January 9, 2026 00:28

Copilot AI review requested due to automatic review settings January 9, 2026 00:28

Copilot started reviewing on behalf of ericsciple January 9, 2026 00:29 View session

Copilot AI reviewed Jan 9, 2026

View reviewed changes

src/git-source-provider.ts Show resolved Hide resolved

ericsciple force-pushed the users/erisciple/26-01-tags branch from 849c4d7 to dc12339 Compare January 9, 2026 01:45

TingluoHuang reviewed Jan 9, 2026

View reviewed changes

src/ref-helper.ts Show resolved Hide resolved

TingluoHuang approved these changes Jan 9, 2026

View reviewed changes

ericsciple merged commit de0fac2 into main Jan 9, 2026
12 checks passed

ericsciple deleted the users/erisciple/26-01-tags branch January 9, 2026 19:42

This was referenced Jan 9, 2026

fetch-tags: true doesn't actually fetch any tags #1471

Open

Preserve tag annotations #290

Open

Can't fetch with fetch-tags when triggered by tag #1467

Open

github-actions bot mentioned this pull request Jan 22, 2026

chore(deps): update actions/checkout action to v6.0.2 koki-develop/clive#206

Merged

1 task

This was referenced Jan 22, 2026

chore(deps): update actions/checkout action to v6.0.2 koki-develop/claude-renovate-review#36

Open

chore(deps): update patch updates anoriqq/pj-tmpl-go#139

Open

taloonjulius28 mentioned this pull request Jan 23, 2026

.github/workflows/test.yml #2363

Closed

github-actions bot mentioned this pull request Jan 25, 2026

chore(deps): update actions/checkout action to v6.0.2 koki-develop/git-aicommit#54

Open

1 task

htunnicliff mentioned this pull request Jan 26, 2026

fetch-tags argument is incorrect #2309

Open

github-actions bot mentioned this pull request Jan 28, 2026

chore(deps): update actions/checkout action to v6.0.2 koki-develop/gat#254

Open

1 task

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Fix tag handling: preserve annotations and explicit fetch-tags #2356

Fix tag handling: preserve annotations and explicit fetch-tags #2356

ericsciple commented Jan 8, 2026 •

edited

Loading

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

ben-manes commented Jan 11, 2026

Uh oh!

ericsciple commented Jan 22, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Fix tag handling: preserve annotations and explicit fetch-tags #2356

Fix tag handling: preserve annotations and explicit fetch-tags #2356

Conversation

ericsciple commented Jan 8, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Issues Fixed

1. fetch-tags: true doesn't fetch tags (#1471)

2. Annotated tags converted to lightweight (#290)

3. Fatal error with tag + fetch-tags: true (#1467)

Testing

Unit Tests

E2E Tests

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

ben-manes commented Jan 11, 2026

Uh oh!

ericsciple commented Jan 22, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

ericsciple commented Jan 8, 2026 •

edited

Loading

1. `fetch-tags: true` doesn't fetch tags (#1471)

3. Fatal error with tag + `fetch-tags: true` (#1467)