Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add documentation for EnterMultipleScope #959

Merged
merged 3 commits into from
Mar 8, 2025
Merged

Conversation

manfred-brands
Copy link
Member

Fixes #958

Copy link

github-actions bot commented Aug 5, 2024

@OsirisTerje
Copy link
Member

Suggest you add some comments on the difference between the old and new way (and thus benefits of the new way) , including the disposables, and also point people to the fact there is no longer an action there.

@OsirisTerje
Copy link
Member

@manfred-brands @SeanKilleen Could we manage to get this in ?
I see there are some code issues here.

@OsirisTerje OsirisTerje changed the title WIP: Add documentation for EnterMultipleScope Add documentation for EnterMultipleScope Mar 8, 2025
@OsirisTerje
Copy link
Member

Just fixed a few comments - due time to get it in, thanks for the writeup @manfred-brands !

@OsirisTerje OsirisTerje marked this pull request as ready for review March 8, 2025 11:00
@OsirisTerje OsirisTerje merged commit dec3493 into master Mar 8, 2025
7 checks passed
@OsirisTerje OsirisTerje deleted the 958_EnterMultipleScope branch March 8, 2025 11:00
github-actions bot pushed a commit that referenced this pull request Mar 8, 2025
Add documentation for EnterMultipleScope dec3493
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.

Document Assert.EnterMultipleScope
3 participants