Skip to content

Conversation

@stephentoub
Copy link
Member

Fixes #77955

@ghost
Copy link

ghost commented Feb 28, 2023

Note regarding the new-api-needs-documentation label:

This serves as a reminder for when your PR is modifying a ref *.cs file and adding/modifying public APIs, to please make sure the API implementation in the src *.cs file is documented with triple slash comments, so the PR reviewers can sign off that change.

@ghost ghost assigned stephentoub Feb 28, 2023
@ghost
Copy link

ghost commented Feb 28, 2023

Tagging subscribers to this area: @dotnet/area-system-memory
See info in area-owners.md if you want to be subscribed.

Issue Details

Fixes #77955

Author: stephentoub
Assignees: stephentoub
Labels:

area-System.Memory, new-api-needs-documentation

Milestone: -

@stephentoub
Copy link
Member Author

All failures are known.

@stephentoub stephentoub merged commit dfa79c8 into dotnet:main Mar 2, 2023
@stephentoub stephentoub deleted the asspanindexrange branch March 2, 2023 14:22
@jeffhandley jeffhandley added the blog-candidate Completed PRs that are candidate topics for blog post coverage label Mar 14, 2023
@ghost ghost locked as resolved and limited conversation to collaborators Apr 13, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-System.Memory blog-candidate Completed PRs that are candidate topics for blog post coverage new-api-needs-documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[API Proposal]: MemoryExtensions.AsSpan(this string? text, Range range)

3 participants