Unit Test provided
Deprecated code removed
Commented code removed
Syntax & Formatting is correct
Is the approach to the problem appropriate?
Are there enough comments inline with the code?
Does the PR follow development standards?
Are the naming conventions appropriate?
Are there any breaking changes which might impact cli-adapters? Is it notified to plugin dev team?
Property files updated?
Authorization has been implemented across these changes
Injection has been prevented (parameterized queries, no eval or system calls)
Any web UI is escaping output (to prevent XSS)
Sensitive data has been identified and is being protected properly
This section lists any additional steps to deploy the PR
Last updated 1 year ago