Skip to content

Comments

UUM-00000: Update CI for release/2.10 branch#1102

Merged
sim-bz merged 3 commits intorelease/2.10from
update_ci_for_release_2.10
Feb 23, 2026
Merged

UUM-00000: Update CI for release/2.10 branch#1102
sim-bz merged 3 commits intorelease/2.10from
update_ci_for_release_2.10

Conversation

@windxu88
Copy link
Collaborator

[Delete any line or section that does not apply]

Purpose of this PR

Update CI for release/2.10 branch to include the latest Editor versions

[JIRA issue. Desc of feature/change. Links to screenshots, design docs, user docs, etc. Remember reviewers may be outside your team, and not know your feature/area that should be explained more.]

Testing status

[Explanation of what’s tested, how tested and existing or new automation tests. Can include manual testing by self and/or QA. Specify test plans. Rarely acceptable to have no testing.]

  • Added an automated test
  • Passed all automated tests
  • Manually tested

Documentation status

[Overview of how documentation is affected by this change. If there is no effect on documentation, explain why. Otherwise, state which sections are changed and why.]

  • Updated CHANGELOG
  • Updated README (if applicable)
  • Commented all public classes, properties, and methods
  • Updated user documentation

Technical risk

[Overall product level assessment of risk of change. Need technical risk & halo effect.]

Comments to reviewers

[Info per person for what to focus on, or historical info to understand who have previously reviewed and coverage. Help them get context.]

Package version

[Justification for updating either the patch, minor, or major version according to the semantic versioning rules]

  • Updated package version

@windxu88 windxu88 requested a review from glabute as a code owner February 23, 2026 19:54
@u-pr
Copy link
Contributor

u-pr bot commented Feb 23, 2026

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪

The PR is small and primarily involves updating CI configurations, removing deprecated packages from manifests, and bumping a .NET SDK version, requiring minimal review effort.
🏅 Score: 95

The changes are clean, straightforward, and fulfill the intended goal, though there is a minor risk of a runtime exception from an unprotected dictionary access.
🧪 No relevant tests
🔒 No security concerns identified
⚡ Recommended focus areas for review

Possible KeyNotFoundException

Accessing the EditorVersionToBranches dictionary directly with an indexer may throw a KeyNotFoundException if the editorVersion is not mapped. Consider using .TryGetValue() to gracefully handle missing versions, or provide a descriptive error message to make potential CI failures easier to troubleshoot.

var branch = settings.Wrench.EditorVersionToBranches[editorVersion];
  • Update review

🤖 Helpful? Please react with 👍/👎 | Questions❓Please reach out in Slack #ask-u-pr

@windxu88 windxu88 requested a review from sim-bz February 23, 2026 19:55
@u-pr
Copy link
Contributor

u-pr bot commented Feb 23, 2026

PR Code Suggestions ✨

No code suggestions found for the PR.

@windxu88 windxu88 changed the title Update CI for release/2.10 branch UUM-00000: Update CI for release/2.10 branch Feb 23, 2026
@sim-bz sim-bz merged commit 0f47dae into release/2.10 Feb 23, 2026
79 of 80 checks passed
@sim-bz sim-bz deleted the update_ci_for_release_2.10 branch February 23, 2026 21:13
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