Skip to content

Fix Winget PR description version wording - #267

Merged
setoelkahfi merged 1 commit into
developmentfrom
fix/winget-pr-description-version
Oct 27, 2026
Merged

setoelkahfi merged 1 commit into
developmentfrom
fix/winget-pr-description-version

Conversation

@setoelkahfi

Copy link
Copy Markdown
Collaborator

Fixes #262

The Winget PR description said "Updates getSigit.siGitCode to version 1.7.5 from v1.7.5", which repeated the tag instead of pointing at the release notes. The workflow now writes the version and a link to the GitHub release:

  • Adds getSigit.siGitCode version 1.7.5. Release notes: <release URL>
  • Updates getSigit.siGitCode to version 1.7.5. Release notes: <release URL>

The tag is no longer part of the sentence, so the version reads as a version. The release link comes from the same tag the workflow already resolves, turned into https://github.com/getsigit/sigit/releases/tag/<tag>.

Only the description changed. The issue-first flow and the two-token setup (WINGET_ISSUE_TOKEN for the issue, WINGET_PR_TOKEN for the PR) are untouched.

cargo fmt -- --check, cargo clippy --tests -- -D warnings and cargo test --locked pass. The workflow YAML parses.

The generated description repeated the version ("to version 1.7.5 from
v1.7.5") because the tag was rendered next to the version. Point the
description at the release notes instead.

Co-Authored-By: siGit Code v1.7.5-acp <noreply@sigit.si>
@setoelkahfi setoelkahfi added this to the v1.7.6 milestone Oct 26, 2026
@sigit-code-review

Copy link
Copy Markdown

siGit Code Review couldn't finish this review: the model service was unavailable. Mention @sigit-code-review to try again.


Automated review by siGit Code Review · commit 83ba964 · see the review dashboard

@setoelkahfi setoelkahfi self-assigned this Oct 26, 2026
@setoelkahfi
setoelkahfi requested a review from paydii October 26, 2026 19:57
@setoelkahfi
setoelkahfi merged commit a0b423b into development Oct 27, 2026
9 checks passed
@setoelkahfi
setoelkahfi deleted the fix/winget-pr-description-version branch October 27, 2026 07:20
@keypair34 keypair34 mentioned this pull request Oct 27, 2026
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.

2 participants