mirror of
https://github.com/community-scripts/ProxmoxVE.git
synced 2026-07-17 19:35:08 +02:00
Compare commits
6 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
| fb610981eb | |||
| a51e1f37f5 | |||
| 8655282c2d | |||
| e15db754a6 | |||
| 772430de7e | |||
| 04a84f5052 |
+163
@@ -0,0 +1,163 @@
|
|||||||
|
name: Close PRs Missing Template
|
||||||
|
|
||||||
|
on:
|
||||||
|
pull_request_target:
|
||||||
|
branches: ["main"]
|
||||||
|
types: [opened, edited, reopened, synchronize, labeled]
|
||||||
|
|
||||||
|
jobs:
|
||||||
|
validate-pr-template:
|
||||||
|
if: github.repository == 'community-scripts/ProxmoxVE'
|
||||||
|
runs-on: ubuntu-latest
|
||||||
|
permissions:
|
||||||
|
pull-requests: write
|
||||||
|
issues: write
|
||||||
|
contents: read
|
||||||
|
steps:
|
||||||
|
- name: Close PR if it does not follow the PR template
|
||||||
|
uses: actions/github-script@v7
|
||||||
|
with:
|
||||||
|
script: |
|
||||||
|
const pr = context.payload.pull_request;
|
||||||
|
const prNumber = pr.number;
|
||||||
|
const author = pr.user.login;
|
||||||
|
const owner = context.repo.owner;
|
||||||
|
const repo = context.repo.repo;
|
||||||
|
|
||||||
|
const allowedBots = [
|
||||||
|
"push-app-to-main[bot]",
|
||||||
|
"push-app-to-main",
|
||||||
|
"community-scripts-pr-app",
|
||||||
|
"github-actions[bot]",
|
||||||
|
"dependabot[bot]",
|
||||||
|
];
|
||||||
|
|
||||||
|
if (allowedBots.includes(author) || author.endsWith("[bot]")) {
|
||||||
|
core.info(`PR #${prNumber} by bot "${author}" — skipping template validation.`);
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
const association = pr.author_association;
|
||||||
|
const exemptAssociations = ["OWNER", "MEMBER", "COLLABORATOR"];
|
||||||
|
if (exemptAssociations.includes(association)) {
|
||||||
|
core.info(`PR #${prNumber} by ${association} "${author}" — skipping template validation.`);
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
const labels = pr.labels.map((label) => label.name);
|
||||||
|
const skipLabels = ["automated pr", "keep open"];
|
||||||
|
|
||||||
|
if (skipLabels.some((label) => labels.includes(label))) {
|
||||||
|
core.info(`PR #${prNumber} has a skip label (${labels.join(", ")}) — skipping template validation.`);
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (pr.draft) {
|
||||||
|
core.info(`PR #${prNumber} is a draft — skipping template validation.`);
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
const body = pr.body || "";
|
||||||
|
const failures = [];
|
||||||
|
|
||||||
|
const requiredSections = [
|
||||||
|
"## ✍️ Description",
|
||||||
|
"## ✅ Prerequisites",
|
||||||
|
"## 🛠️ Type of Change",
|
||||||
|
];
|
||||||
|
|
||||||
|
for (const section of requiredSections) {
|
||||||
|
if (!body.includes(section)) {
|
||||||
|
failures.push(`Missing required section: \`${section}\``);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
const descriptionMatch = body.match(
|
||||||
|
/## ✍️ Description\s*\n+([\s\S]*?)(?=\n## )/i
|
||||||
|
);
|
||||||
|
const description = (descriptionMatch?.[1] || "").trim();
|
||||||
|
if (!description) {
|
||||||
|
failures.push("The **Description** section is empty.");
|
||||||
|
}
|
||||||
|
|
||||||
|
const prerequisiteCheckboxes = [
|
||||||
|
"**Self-review completed**",
|
||||||
|
"**Tested thoroughly**",
|
||||||
|
"**No security risks**",
|
||||||
|
];
|
||||||
|
|
||||||
|
for (const checkbox of prerequisiteCheckboxes) {
|
||||||
|
const escaped = checkbox.replace(/([.*+?^=!:${}()|[\]\/\\])/g, "\\$1");
|
||||||
|
const regex = new RegExp(`- \\[(x|X)\\]\\s*${escaped}`, "i");
|
||||||
|
if (!regex.test(body)) {
|
||||||
|
failures.push(`Prerequisite not checked: ${checkbox}`);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
const typeOfChangeCheckboxes = [
|
||||||
|
"🐞 **Bug fix**",
|
||||||
|
"✨ **New feature**",
|
||||||
|
"💥 **Breaking change**",
|
||||||
|
"🆕 **New script**",
|
||||||
|
"🌍 **Website update**",
|
||||||
|
"🔧 **Refactoring / Code Cleanup**",
|
||||||
|
"📝 **Documentation update**",
|
||||||
|
];
|
||||||
|
|
||||||
|
const hasTypeChecked = typeOfChangeCheckboxes.some((checkbox) => {
|
||||||
|
const escaped = checkbox.replace(/([.*+?^=!:${}()|[\]\/\\])/g, "\\$1");
|
||||||
|
const regex = new RegExp(`- \\[(x|X)\\]\\s*${escaped}`, "i");
|
||||||
|
return regex.test(body);
|
||||||
|
});
|
||||||
|
|
||||||
|
if (!hasTypeChecked) {
|
||||||
|
failures.push("At least one **Type of Change** checkbox must be checked.");
|
||||||
|
}
|
||||||
|
|
||||||
|
if (failures.length === 0) {
|
||||||
|
core.info(`PR #${prNumber} follows the PR template.`);
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
core.info(`Closing PR #${prNumber} — template validation failed.`);
|
||||||
|
|
||||||
|
const templateUrl =
|
||||||
|
"https://github.com/community-scripts/ProxmoxVE/blob/main/.github/pull_request_template.md";
|
||||||
|
const failureList = failures.map((item) => `- ${item}`).join("\n");
|
||||||
|
|
||||||
|
const comment = [
|
||||||
|
`👋 Hi @${author},`,
|
||||||
|
``,
|
||||||
|
`This pull request was closed because it does not follow the [PR template](${templateUrl}).`,
|
||||||
|
``,
|
||||||
|
`Please fix the following and open a new PR (or reopen this one after updating the description):`,
|
||||||
|
``,
|
||||||
|
failureList,
|
||||||
|
``,
|
||||||
|
`> Use the template sections, fill in the description, check all prerequisite boxes, and select at least one type of change.`,
|
||||||
|
``,
|
||||||
|
`Maintainers can add the \`keep open\` label to exempt a PR from this check.`,
|
||||||
|
``,
|
||||||
|
`Thank you for contributing! 🙏`,
|
||||||
|
].join("\n");
|
||||||
|
|
||||||
|
await github.rest.issues.createComment({
|
||||||
|
owner,
|
||||||
|
repo,
|
||||||
|
issue_number: prNumber,
|
||||||
|
body: comment,
|
||||||
|
});
|
||||||
|
|
||||||
|
await github.rest.pulls.update({
|
||||||
|
owner,
|
||||||
|
repo,
|
||||||
|
pull_number: prNumber,
|
||||||
|
state: "closed",
|
||||||
|
});
|
||||||
|
|
||||||
|
await github.rest.issues.addLabels({
|
||||||
|
owner,
|
||||||
|
repo,
|
||||||
|
issue_number: prNumber,
|
||||||
|
labels: ["missing pr template"],
|
||||||
|
});
|
||||||
+10
-1
@@ -502,11 +502,20 @@ Exercise vigilance regarding copycat or coat-tailing sites that seek to exploit
|
|||||||
|
|
||||||
</details>
|
</details>
|
||||||
|
|
||||||
|
## 2026-07-17
|
||||||
|
|
||||||
|
### 🧰 Tools
|
||||||
|
|
||||||
|
- #### ✨ New Features
|
||||||
|
|
||||||
|
- update-lxc: autoremove and autoclean after apt full-upgrade [@soupy-boy](https://github.com/soupy-boy) ([#15831](https://github.com/community-scripts/ProxmoxVE/pull/15831))
|
||||||
|
|
||||||
## 2026-07-16
|
## 2026-07-16
|
||||||
|
|
||||||
### 🆕 New Scripts
|
### 🆕 New Scripts
|
||||||
|
|
||||||
- Beaverhabits ([#15813](https://github.com/community-scripts/ProxmoxVE/pull/15813))
|
- Sync-In ([#15812](https://github.com/community-scripts/ProxmoxVE/pull/15812))
|
||||||
|
- Beaverhabits ([#15813](https://github.com/community-scripts/ProxmoxVE/pull/15813))
|
||||||
- Notediscovery ([#15811](https://github.com/community-scripts/ProxmoxVE/pull/15811))
|
- Notediscovery ([#15811](https://github.com/community-scripts/ProxmoxVE/pull/15811))
|
||||||
|
|
||||||
### 🚀 Updated Scripts
|
### 🚀 Updated Scripts
|
||||||
|
|||||||
@@ -28,6 +28,7 @@ function update_script() {
|
|||||||
msg_error "No ${APP} Installation Found!"
|
msg_error "No ${APP} Installation Found!"
|
||||||
exit
|
exit
|
||||||
fi
|
fi
|
||||||
|
ensure_dependencies libusb-1.0-0
|
||||||
|
|
||||||
msg_info "Stopping Service"
|
msg_info "Stopping Service"
|
||||||
systemctl stop esphome-device-builder 2>/dev/null || true
|
systemctl stop esphome-device-builder 2>/dev/null || true
|
||||||
|
|||||||
@@ -14,7 +14,8 @@ network_check
|
|||||||
update_os
|
update_os
|
||||||
|
|
||||||
msg_info "Installing Dependencies"
|
msg_info "Installing Dependencies"
|
||||||
$STD apt install -y git
|
$STD apt install -y git \
|
||||||
|
libusb-1.0-0
|
||||||
msg_ok "Installed Dependencies"
|
msg_ok "Installed Dependencies"
|
||||||
|
|
||||||
PYTHON_VERSION="3.12" setup_uv
|
PYTHON_VERSION="3.12" setup_uv
|
||||||
|
|||||||
@@ -78,7 +78,7 @@ function update_container() {
|
|||||||
alpine) pct exec "$container" -- ash -c "apk -U upgrade" ;;
|
alpine) pct exec "$container" -- ash -c "apk -U upgrade" ;;
|
||||||
archlinux) pct exec "$container" -- bash -c "pacman -Syyu --noconfirm" ;;
|
archlinux) pct exec "$container" -- bash -c "pacman -Syyu --noconfirm" ;;
|
||||||
fedora | rocky | centos | alma) pct exec "$container" -- bash -c "dnf -y update && dnf -y upgrade" ;;
|
fedora | rocky | centos | alma) pct exec "$container" -- bash -c "dnf -y update && dnf -y upgrade" ;;
|
||||||
ubuntu | debian | devuan) pct exec "$container" -- bash -c "apt-get update 2>/dev/null | grep 'packages.*upgraded'; apt list --upgradable 2>/dev/null | cat && apt-get -yq dist-upgrade 2>&1; rm -rf /usr/lib/python3.*/EXTERNALLY-MANAGED || true" ;;
|
ubuntu | debian | devuan) pct exec "$container" -- bash -c "apt-get update 2>/dev/null | grep 'packages.*upgraded'; apt list --upgradable 2>/dev/null | cat && apt-get -yq dist-upgrade 2>&1; apt-get -yq autoremove 2>&1; apt-get -yq autoclean 2>&1; rm -rf /usr/lib/python3.*/EXTERNALLY-MANAGED || true" ;;
|
||||||
opensuse) pct exec "$container" -- bash -c "zypper ref && zypper --non-interactive dup" ;;
|
opensuse) pct exec "$container" -- bash -c "zypper ref && zypper --non-interactive dup" ;;
|
||||||
esac
|
esac
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user