Skip to content

WD-32245: Add security considerations section to PR template#5530

Merged
bartaz merged 1 commit intomainfrom
bartaz-patch-1
Dec 12, 2025
Merged

WD-32245: Add security considerations section to PR template#5530
bartaz merged 1 commit intomainfrom
bartaz-patch-1

Conversation

@bartaz
Copy link
Copy Markdown
Member

@bartaz bartaz commented Dec 12, 2025

Added a section for security considerations in the PR template.

Issue / Card

Fixes WD-32245

Security

  • Security considerations for review (list them):
  • This PR has no security considerations (explain why): change in PR template .md file

Added a section for security considerations in the PR template.
Copilot AI review requested due to automatic review settings December 12, 2025 13:14
@webteam-app
Copy link
Copy Markdown

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds a new "Security" section to the pull request template, positioned between "Testing" and "Issue / Card" sections. The addition encourages contributors to explicitly consider and document security implications of their changes.

Key changes:

  • Adds a "Security" section with checkboxes for listing security considerations or explaining why there are none
  • Provides example security considerations covering access control, input validation, and sensitive data exposure

Comment thread .github/pull_request_template.md
Copy link
Copy Markdown
Contributor

@steverydz steverydz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@bartaz bartaz merged commit 0fabc7d into main Dec 12, 2025
19 checks passed
@bartaz bartaz deleted the bartaz-patch-1 branch December 12, 2025 13:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants