Skip to content

Use ValueTask instead of Task in Assertion logic to reduce allocations#1859

Merged
thomhurst merged 7 commits into
thomhurst:mainfrom
AnnaSasDev:valuetask
Feb 15, 2025
Merged

Use ValueTask instead of Task in Assertion logic to reduce allocations#1859
thomhurst merged 7 commits into
thomhurst:mainfrom
AnnaSasDev:valuetask

Conversation

@AnnaSasDev
Copy link
Copy Markdown
Contributor

A priliminary search of which Task methods can be replaced by ValueTask
#1850

@thomhurst thomhurst changed the title ValueTask instead of Task Use ValueTask instead of Task in Assertion logic to reduce allocations Feb 15, 2025
Comment thread TUnit.Assertions/AssertConditions/AssertionResult.cs
@thomhurst
Copy link
Copy Markdown
Owner

All looks good! Could you update the doc examples too? 😇

@thomhurst
Copy link
Copy Markdown
Owner

Thanks @AnnaSasDev !

@thomhurst thomhurst disabled auto-merge February 15, 2025 22:08
@thomhurst thomhurst merged commit 4f7ac05 into thomhurst:main Feb 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants