ProxmoxVED/.github/pull_request_template.md
2025-03-14 11:00:51 +01:00

1.7 KiB
Raw Permalink Blame History

🛑 New scripts must first be submitted to ProxmoxVED for testing.
PRs for new scripts that skip this process will be closed.


✍️ Description

Link: #

Prerequisites (X in brackets)

  • Self-review completed Code follows project standards.
  • Tested thoroughly Changes work as expected.
  • No breaking changes Existing functionality remains intact.
  • No security risks No hardcoded secrets, unnecessary privilege escalations, or permission issues.

🛠️ Type of Change (X in brackets)

  • 🐞 Bug fix Resolves an issue without breaking functionality.
  • New feature Adds new, non-breaking functionality.
  • 💥 Breaking change Alters existing functionality in a way that may require updates.
  • 🆕 New script A fully functional and tested script or script set.
  • 🌍 Website update Changes to website-related JSON files or metadata.
  • 🔧 Refactoring / Code Cleanup Improves readability or maintainability without changing functionality.
  • 📝 Documentation update Changes to README, AppName.md, CONTRIBUTING.md, or other docs.

🔍 Code & Security Review (X in brackets)

  • Follows Code_Audit.md & CONTRIBUTING.md guidelines
  • Uses correct script structure (AppName.sh, AppName-install.sh, AppName.json)
  • No hardcoded credentials

📋 Additional Information (optional)