-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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 CustomAttrbute #9175
Conversation
Learn Build status updates of commit d338d34: ✅ Validation status: passedFor more details, please refer to the build report. For any questions, please:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Commenting to give @gewarren final review / merge decision.
I've read through the code and the text, and I think it's good to merge.
snippets/csharp/System.Reflection.Metadata/CustomAttribute/Overview/CustomAttributeSnippets.cs
Outdated
Show resolved
Hide resolved
Co-authored-by: Genevieve Warren <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
Learn Build status updates of commit 7bdcaca: ✅ Validation status: passedFor more details, please refer to the build report. For any questions, please:
|
Learn Build status updates of commit feb6a87: ✅ Validation status: passedFor more details, please refer to the build report. For any questions, please:
|
Thanks @MSDN-WhiteKnight! |
Add remarks and example for System.Reflection.Metadata.CustomAttrbute