Skip to content

#945509 Support ref readonly parameters#712

Merged
bianyifan merged 5 commits into
developfrom
nickwalk/refreadonlyparams
Oct 30, 2025
Merged

#945509 Support ref readonly parameters#712
bianyifan merged 5 commits into
developfrom
nickwalk/refreadonlyparams

Conversation

@nickwalkmsft

Copy link
Copy Markdown
Collaborator

Work item: https://ceapex.visualstudio.com/Engineering/_workitems/edit/945509

Add support for correct rendering of "ref readonly" parameter in C# method signature documentation.

Added one unit test and also verified output against the case mentioned in the work item (https://learn.microsoft.com/en-us/dotnet/api/system.runtime.compilerservices.unsafe.asref?view=net-8.0#system-runtime-compilerservices-unsafe-asref-1%28-0@%29 should render with scoped ref readonly T source instead of scoped ref T source.

I couldn't get the C++ tests running but they appear to be out of scope here.

Add support for correct rendering of "ref readonly" parameter in C#
method signature documentation.
@nickwalkmsft nickwalkmsft changed the base branch from main to develop September 25, 2024 16:24
@v-fuquanli

Copy link
Copy Markdown
Contributor

@xmdanni

xmdanni commented Nov 15, 2024

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@xmdanni

xmdanni commented Nov 15, 2024

Copy link
Copy Markdown
Collaborator

@bianyifan

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@xmdanni

xmdanni commented Oct 29, 2025

Copy link
Copy Markdown
Collaborator

@bianyifan

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@xmdanni

xmdanni commented Oct 29, 2025

Copy link
Copy Markdown
Collaborator

@bianyifan

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@xmdanni

xmdanni commented Oct 29, 2025

Copy link
Copy Markdown
Collaborator

@bianyifan bianyifan merged commit 6935391 into develop Oct 30, 2025
2 checks passed
@bianyifan bianyifan deleted the nickwalk/refreadonlyparams branch October 30, 2025 02:31
huangmin-ms added a commit that referenced this pull request Oct 30, 2025
Add support for correct rendering of "ref readonly" parameter in C#
method signature documentation.
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.

5 participants