Add conceptual content for partial properties #42177
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Now that `partial` can be used for properties and indexers as well, the name "partial member" is better.
Add the feature spec to the learn platform.
- Update the partial members article to include details for partial properties. This includes new information that also applies to partial methods. - Update the partial types article to include that partial members can now include properties and indexers.
BillWagner
force-pushed
the
partial-properties
branch
from
9878a9e to
25413e7
Compare
Update language reference with links Add links or text on partial properties and indexers in the language reference.
These articles mention partial types. Make necessary updates, clear open issues on these articles.
BillWagner
force-pushed
the
partial-properties
branch
from
25413e7 to
70e18ed
Compare
dotnet-policy-service
bot
added
the
okr-quality
Member
Author
|
The snippets 5000 error is a false positive. This PR moves samples from locations without a project to a location that has a project to compile. |
BillWagner
force-pushed
the
partial-properties
branch
from
57b578c to
517b292
Compare
dotnet-policy-service
bot
added
the
okr-curation
BillWagner
force-pushed
the
partial-properties
branch
2 times, most recently
from
8175058 to
f1ea86f
Compare
The introductory articles for properties and indexers were largely duplicated, combine them.
BillWagner
force-pushed
the
partial-properties
branch
from
79e56a2 to
8b4402d
Compare
Fix all open issues on the the combined articles. Also, remove unsed snippets
BillWagner
force-pushed
the
partial-properties
branch
from
366eb95 to
9606906
Compare
Edit and freshness pass on the combined indexers article.
BillWagner
force-pushed
the
partial-properties
branch
from
b80f953 to
3c7b15c
Compare
Use clean repo to update all links
IEvangelist
approved these changes
Aug 22, 2024
Member
IEvangelist
left a comment
IEvangelist
left a comment
There was a problem hiding this comment.
LGTM, just had one suggestion.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Contributes to #41572
struct,class, andrecordtypes.requiredkeyword #35166 - Add an example withrequiredproperties.All lot of the updates are small. It may be easier to review commit-by-commit.
Internal previews
Toggle expand/collapse
Note
This table shows preview links for the 30 files with the most changes. For preview links for other files in this PR, select OpenPublishing.Build Details within checks.