prowler-pr
Compare original and translation side by side
🇺🇸
Original
English🇨🇳
Translation
ChinesePR Creation Process
PR创建流程
- Analyze changes: to understand ALL commits
git diff main...HEAD - Determine affected components: SDK, API, UI, MCP, Docs
- Fill template sections based on changes
- Create PR with
gh pr create
- 分析变更:使用了解所有提交记录
git diff main...HEAD - 确定受影响的组件:SDK、API、UI、MCP、Docs
- 根据变更内容填写模板章节
- 使用创建PR
gh pr create
PR Template Structure
PR模板结构
markdown
undefinedmarkdown
undefinedContext
Context
{Why this change? Link issues with }
Fix #XXXX{Why this change? Link issues with }
Fix #XXXXDescription
Description
{Summary of changes and dependencies}
{Summary of changes and dependencies}
Steps to review
Steps to review
{How to test/verify the changes}
{How to test/verify the changes}
Checklist
Checklist
<details>
<summary><b>Community Checklist</b></summary>
- This feature/issue is listed in here or roadmap.prowler.com
- Is it assigned to me, if not, request it via the issue/feature in here or Prowler Community Slack
- Are there new checks included in this PR? Yes / No
- If so, do we need to update permissions for the provider?
- Review if the code is being covered by tests.
- Review if code is being documented following https://github.com/google/styleguide/blob/gh-pages/pyguide.md#38-comments-and-docstrings
- Review if backport is needed.
- Review if is needed to change the Readme.md
- Ensure new entries are added to CHANGELOG.md, if applicable.
<details>
<summary><b>Community Checklist</b></summary>
- This feature/issue is listed in here or roadmap.prowler.com
- Is it assigned to me, if not, request it via the issue/feature in here or Prowler Community Slack
- Are there new checks included in this PR? Yes / No
- If so, do we need to update permissions for the provider?
- Review if the code is being covered by tests.
- Review if code is being documented following https://github.com/google/styleguide/blob/gh-pages/pyguide.md#38-comments-and-docstrings
- Review if backport is needed.
- Review if is needed to change the Readme.md
- Ensure new entries are added to CHANGELOG.md, if applicable.
SDK/CLI
SDK/CLI
- Are there new checks included in this PR? Yes / No
- If so, do we need to update permissions for the provider? Please review this carefully.
- Are there new checks included in this PR? Yes / No
- If so, do we need to update permissions for the provider? Please review this carefully.
UI (if applicable)
UI (if applicable)
- All issue/task requirements work as expected on the UI
- Screenshots/Video - Mobile (X < 640px)
- Screenshots/Video - Tablet (640px > X < 1024px)
- Screenshots/Video - Desktop (X > 1024px)
- Ensure new entries are added to ui/CHANGELOG.md
- All issue/task requirements work as expected on the UI
- Screenshots/Video - Mobile (X < 640px)
- Screenshots/Video - Tablet (640px > X < 1024px)
- Screenshots/Video - Desktop (X > 1024px)
- Ensure new entries are added to ui/CHANGELOG.md
API (if applicable)
API (if applicable)
- All issue/task requirements work as expected on the API
- Endpoint response output (if applicable)
- EXPLAIN ANALYZE output for new/modified queries or indexes (if applicable)
- Performance test results (if applicable)
- Any other relevant evidence of the implementation (if applicable)
- Verify if API specs need to be regenerated.
- Check if version updates are required.
- Ensure new entries are added to api/CHANGELOG.md
- All issue/task requirements work as expected on the API
- Endpoint response output (if applicable)
- EXPLAIN ANALYZE output for new/modified queries or indexes (if applicable)
- Performance test results (if applicable)
- Any other relevant evidence of the implementation (if applicable)
- Verify if API specs need to be regenerated.
- Check if version updates are required.
- Ensure new entries are added to api/CHANGELOG.md
License
License
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
undefinedBy submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
undefinedComponent-Specific Rules
组件特定规则
| Component | CHANGELOG | Extra Checks |
|---|---|---|
| SDK | | New checks → permissions update? |
| API | | API specs, version bump, endpoint output, EXPLAIN ANALYZE, performance |
| UI | | Screenshots for Mobile/Tablet/Desktop |
| MCP | | N/A |
| 组件 | 变更日志文件 | 额外检查项 |
|---|---|---|
| SDK | | 新增检查项 → 是否需要更新权限? |
| API | | API规范、版本升级、端点输出、EXPLAIN ANALYZE、性能测试 |
| UI | | 移动端/平板端/桌面端截图 |
| MCP | | 无 |
Commands
常用命令
bash
undefinedbash
undefinedCheck current branch status
Check current branch status
git status
git log main..HEAD --oneline
git status
git log main..HEAD --oneline
View full diff
View full diff
git diff main...HEAD
git diff main...HEAD
Create PR with heredoc for body
Create PR with heredoc for body
gh pr create --title "feat: description" --body "$(cat <<'EOF'
gh pr create --title "feat: description" --body "$(cat <<'EOF'
Context
Context
...
EOF
)"
...
EOF
)"
Create draft PR
Create draft PR
gh pr create --draft --title "feat: description"
undefinedgh pr create --draft --title "feat: description"
undefinedTitle Conventions
标题规范
Follow conventional commits:
- New feature
feat: - Bug fix
fix: - Documentation
docs: - Maintenance
chore: - Code restructure
refactor: - Tests
test:
遵循Conventional Commits规范:
- 新功能
feat: - 修复Bug
fix: - 文档更新
docs: - 维护工作
chore: - 代码重构
refactor: - 测试相关
test:
Before Creating PR
创建PR前的检查
- ✅ All tests pass locally
- ✅ Linting passes (or component-specific)
make lint - ✅ CHANGELOG updated (if applicable)
- ✅ Branch is up to date with main
- ✅ Commits are clean and descriptive
- ✅ 所有测试在本地运行通过
- ✅ 代码检查通过(或组件特定的检查命令)
make lint - ✅ 已更新CHANGELOG(如适用)
- ✅ 当前分支已与main分支同步
- ✅ 提交记录清晰且描述准确
Resources
参考资源
- Documentation: See references/ for links to local developer guide
- 文档:查看references/获取本地开发者指南的链接