Update PR Template
This commit is contained in:
parent
b95d290e20
commit
bbeb7758d6
42
.github/pull_request_template.md
vendored
42
.github/pull_request_template.md
vendored
@ -1,25 +1,43 @@
|
||||
## ✍️ Description
|
||||
<!-- Provide a clear and concise description of your changes. -->
|
||||
## 🚨 Important Notice for New Scripts
|
||||
🛑 **New scripts must first be submitted to [ProxmoxVED](https://github.com/community-scripts/ProxmoxVED) for testing.**
|
||||
PRs for new scripts that skip this process will be closed.
|
||||
|
||||
## 🔗 Related PR / Discussion / Issue
|
||||
---
|
||||
|
||||
## ✍️ Description
|
||||
<!-- Briefly describe your changes. -->
|
||||
|
||||
## 🔗 Related PR / Issue
|
||||
|
||||
Link: #
|
||||
|
||||
## ✅ Prerequisites
|
||||
## ✅ Prerequisites (**X** in brackets)
|
||||
|
||||
Before this PR can be reviewed, the following must be completed:
|
||||
- [ ] **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.
|
||||
|
||||
- [] **Self-review performed** – Code follows established patterns and conventions.
|
||||
- [] **Testing performed** – Changes have been thoroughly tested and verified.
|
||||
---
|
||||
|
||||
## 🛠️ Type of Change
|
||||
## 🛠️ Type of Change (**X** in brackets)
|
||||
|
||||
Select all that apply:
|
||||
|
||||
- [] 🆕 **New script** – A fully functional and tested script or script set.
|
||||
- [ ] 🐞 **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)
|
||||
<!-- Provide extra context, screenshots, or references if needed. -->
|
||||
<!-- Add any extra context, screenshots, or references. -->
|
||||
|
Loading…
x
Reference in New Issue
Block a user