-
Notifications
You must be signed in to change notification settings - Fork 4k
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
Update DotNet_Security_Cheat_Sheet.md #1352
Conversation
addition to A08 based on Microsoft documentation
Nice edit thank you! I am seeing some lint errors, can you kindly fix this? cheatsheets/DotNet_Security_Cheat_Sheet.md:799 MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- XmlSerializer and DataContra..."] |
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.
Looks good! Just added a missing .
for .NET
and a link to the MS docs
Co-authored-by: Shlomo Zalman Heigh <shlomozalmanheigh@gmail.com>
Co-authored-by: Shlomo Zalman Heigh <shlomozalmanheigh@gmail.com>
I still see one lint error: cheatsheets/DotNet_Security_Cheat_Sheet.md:799 MD032/blanks-around-lists Lists should be surrounded by blank lines [Context: "- XmlSerializer and DataContra..."] |
Lint fixes and @szh 's commits all integrated and merged! |
addition to A08 based on Microsoft documentation
Thank you for submitting a Pull Request (PR) to the Cheat Sheet Series.
Please make sure that for your contribution:
[TEXT](URL)
If your PR is related to an issue, please finish your PR text with the following line:
This PR covers issue #.
Thank you again for your contribution 😃