Skip to content

Comments

fix: update Net4_7 Core API snapshot for merged PRs#4959

Merged
thomhurst merged 1 commit intomainfrom
fix/update-all-snapshots
Feb 19, 2026
Merged

fix: update Net4_7 Core API snapshot for merged PRs#4959
thomhurst merged 1 commit intomainfrom
fix/update-all-snapshots

Conversation

@thomhurst
Copy link
Owner

Summary

Test plan

  • CI passes Core_Library_Has_No_API_Changes on net472 (windows)
  • No regressions on other frameworks

Add Defaults class, SetRetryBackoff method, BackoffMs/BackoffMultiplier
properties on RetryAttribute, RetryBackoffMs/RetryBackoffMultiplier on
TestDetails and ITestConfiguration - matching the DotNet8_0 snapshot
but without DynamicallyAccessedMembers annotations.
@claude
Copy link
Contributor

claude bot commented Feb 19, 2026

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

This is a straightforward snapshot maintenance PR that correctly follows the project's snapshot testing rules:

Good catch on keeping the Net4_7 snapshot in sync with the other framework targets.

@thomhurst thomhurst merged commit ae27a8e into main Feb 19, 2026
13 of 14 checks passed
@thomhurst thomhurst deleted the fix/update-all-snapshots branch February 19, 2026 16:36
This was referenced Feb 22, 2026
This was referenced Feb 23, 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.

1 participant