Skip to content

Use git ls-remote instead of git tag -l for tag check#25281

Merged
EngincanV merged 3 commits into
rel-10.3from
fix/use-ls-remote-for-tag-check
Apr 16, 2026
Merged

Use git ls-remote instead of git tag -l for tag check#25281
EngincanV merged 3 commits into
rel-10.3from
fix/use-ls-remote-for-tag-check

Conversation

@maliming

Copy link
Copy Markdown
Member

Use git ls-remote --tags origin <tag> to query the remote directly instead of git tag -l which requires fetching all tags locally. This avoids the expensive git fetch --tags origin step in the CI workflow.

Copilot AI review requested due to automatic review settings April 16, 2026 06:18

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Optimizes the NuGet package version change detector workflow by avoiding a full tag fetch, and instead checking tag existence directly against the remote.

Changes:

  • Remove git fetch --tags origin from the CI workflow to avoid fetching all tags.
  • Update the tag existence check in .github/scripts/update_dependency_changes.py to use git ls-remote --tags origin <tag>.

Reviewed changes

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

File Description
.github/workflows/nuget-packages-version-change-detector.yml Stops fetching all tags; only fetches the base branch needed for diffs.
.github/scripts/update_dependency_changes.py Switches tag existence check from local tags to querying the remote via git ls-remote.

Comment thread .github/scripts/update_dependency_changes.py Outdated

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

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

Comment thread .github/scripts/update_dependency_changes.py Outdated
@maliming maliming requested a review from Copilot April 16, 2026 06:27

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

@maliming maliming requested a review from EngincanV April 16, 2026 06:33
@EngincanV EngincanV merged commit 366c257 into rel-10.3 Apr 16, 2026
6 checks passed
@EngincanV EngincanV deleted the fix/use-ls-remote-for-tag-check branch April 16, 2026 07:39
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.

3 participants