Skip to content
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

Code Security Documentation Content Update #27406

Merged
merged 8 commits into from
Jan 31, 2025

Conversation

kassenq
Copy link
Collaborator

@kassenq kassenq commented Jan 31, 2025

What does this PR do? What is the motivation?

  • Clarifying confusing sections of the docs - when we merged Code Analysis docs with Application Security, there were some things that slipped through the cracks

Merge instructions

Merge after approval

Merge readiness:

  • [ x ] Ready for merge

Merge queue is enabled in this repo. To have it automatically merged after it receives the required reviews, create the PR (from a branch that follows the <yourname>/description naming convention) and then add the following PR comment:

/merge

Additional notes

@kassenq kassenq requested a review from a team as a code owner January 31, 2025 22:11
Copy link
Contributor

Preview links (active after the build_preview check completes)

Modified Files

@aliciascott aliciascott self-assigned this Jan 31, 2025
@aliciascott
Copy link
Contributor

Created DOCS card for review

@aliciascott aliciascott added the editorial review Waiting on a more in-depth review label Jan 31, 2025
@michaelcretzman michaelcretzman removed the editorial review Waiting on a more in-depth review label Jan 31, 2025
@michaelcretzman
Copy link
Contributor

LGTM

@michaelcretzman michaelcretzman merged commit 8fad7af into master Jan 31, 2025
14 of 15 checks passed
@michaelcretzman michaelcretzman deleted the kassenq/code-security-content-update branch January 31, 2025 23:16
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.

3 participants