Repository navigation
Point the release notes page at the change history page - #5616
Merged
Gabriel Dufresne (GabrielDuf) merged 3 commits intoOct 28, 2026
Merged
Conversation
Contributor
There was a problem hiding this comment.
🟢 Approval recommended
The two-line change matches the intended scope, preserves shared URL usage, and has no unresolved findings.
No open issues 🎉
What changed in this PR
Updates UniGetUI’s shared release-notes URL to the Devolutions change-history page, redirecting existing in-app and TUI consumers without changing their code.
Changes:
- Points
CoreData.ReleaseNotesUrlto the new change-history URL. - Updates the test expectation while retaining the assertion that
GetGitHubReleasePageUrl()uses the shared constant.
| File | Description |
|---|---|
src/UniGetUI.Core.Data/CoreData.cs |
Changes the shared release-notes destination. |
src/UniGetUI.Core.Data.Tests/CoreTests.cs |
Updates the expected URL in the existing test. |
Review effort: Balanced
![randy-but-a-ro[bot]](https://avatars.githubusercontent.com/in/113084?s=60&v=4){kind=small}
There was a problem hiding this comment.
🤖 Pull request was approved automatically: the AI review is complete and all its review threads are resolved. 🎉
Integration Details
{
"deliveryId": "3513af00-d238-11f1-9661-c937653279ba",
"headSha": "68d5e25c2fafff282d74e8374bafd5188b164d1e",
"reviewer": "copilot-pull-request-reviewer[bot]"
}
…cked ToolbarSelectionStateTests deletes its temp folder about 15 ms after writing catalog.json. Another process (most likely an antivirus scan of the new file) sometimes still holds it, so Dispose threw an IOException and failed the test even though every assertion passed. That failed the .NET Tests check on this PR twice and 3 of 10 local full-assembly runs. Dispose now retries the delete for about a second and then leaves the temp folder behind rather than failing. 10 of 10 local runs pass. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
There was a problem hiding this comment.
🤖 Pull request was approved automatically: the AI review is complete and all its review threads are resolved. 🎉
Integration Details
{
"deliveryId": "efb21d80-d245-11f1-89db-ec83a8e7105a",
"headSha": "1c319452b9181fe49b8a7a19646e63dd8af064e5",
"reviewer": "copilot-pull-request-reviewer[bot]"
}
Gabriel Dufresne (GabrielDuf)
deleted the
chore/release-notes-change-history
branch
October 28, 2026 12:26
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The in-app Release notes page, the TUI help and main-window links, and
CoreData.GetGitHubReleasePageUrl()now open https://devolutions.net/data/UniGetUIChangeHistoryUpdate.htm instead of https://devolutions.net/unigetui/release-notes/. All of them read the singleCoreData.ReleaseNotesUrlconstant; its test is updated to match. The Chocolatey nuspec<releaseNotes>link is left on the existing page.Also merges main and makes
ToolbarSelectionStateTests(from #5614) retry its temp-folder cleanup: another process briefly held the freshly writtencatalog.json, soDisposethrew and failed .NET Tests on this PR twice (3/10 local runs; 10/10 pass with the retry).🤖 Generated with Claude Code