[PR #1040] [MERGED] Add scripts for automated development contribution #2979

Closed
opened 2026-02-26 13:32:00 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/community-scripts/ProxmoxVE/pull/1040
Author: @quantumryuu
Created: 12/27/2024
Status: Merged
Merged: 12/27/2024
Merged by: @michelroegl-brunner

Base: new_script_testingHead: contribute


📝 Commits (10+)

📊 Changes

12 files changed (+1057 additions, -14 deletions)

View changed files

.github/CONTRIBUTING.md (+0 -14)
.github/CONTRIBUTOR_GUIDE/CONTRIBUTING.md (+108 -0)
.github/CONTRIBUTOR_GUIDE/ct/AppName.md (+231 -0)
.github/CONTRIBUTOR_GUIDE/ct/AppName.sh (+93 -0)
.github/CONTRIBUTOR_GUIDE/dev-scripts/README.md (+42 -0)
.github/CONTRIBUTOR_GUIDE/dev-scripts/change_lxc.sh (+52 -0)
.github/CONTRIBUTOR_GUIDE/dev-scripts/cleanup_lxc.sh (+60 -0)
.github/CONTRIBUTOR_GUIDE/dev-scripts/new_lxc.sh (+84 -0)
.github/CONTRIBUTOR_GUIDE/install/AppName-install.md (+262 -0)
.github/CONTRIBUTOR_GUIDE/install/AppName-install.sh (+86 -0)
.github/CONTRIBUTOR_GUIDE/json/AppName.json (+34 -0)
.github/CONTRIBUTOR_GUIDE/json/AppName.md (+5 -0)

📄 Description

🛠️ Note:
We are meticulous about merging code into the main branch, so please understand that pull requests not meeting the project's standards may be rejected. It's never personal!
🎮 Note for game-related scripts: These have a lower likelihood of being merged.


✍️ Description

Please see everything inside the dev-scripts folder!
I have a readme file with the breakdown and steps needed for the scripts.
Essentially what these scripts do is to setup a branch for creating a new helper script and automate a couple of things that may be forgotten like changing the build.func, install.func etc to point to the new branch instead of the official repo!
@michelroegl-brunner there you go mate, they're rough around the edge since they were developed for personal use but the workflow works! Although change_lxc.sh and cleanup_lxc.sh need a bit of more testing, new_lxc.sh seems to work for my needs pretty good (tested with 3-4 new helper scripts that I created a PR for them!)


Please remove unneeded lines!


🛠️ Type of Change

Please check the relevant options:

  • Bug fix (non-breaking change that resolves an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change unexpectedly)
  • New script (a fully functional and thoroughly tested script or set of scripts)

Prerequisites

The following steps must be completed for the pull request to be considered:

  • Self-review performed (I have reviewed my code to ensure it follows established patterns and conventions.)
  • Testing performed (I have thoroughly tested my changes and verified expected functionality.)
  • Documentation updated (I have updated any relevant documentation)

📋 Additional Information (optional)


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/community-scripts/ProxmoxVE/pull/1040 **Author:** [@quantumryuu](https://github.com/quantumryuu) **Created:** 12/27/2024 **Status:** ✅ Merged **Merged:** 12/27/2024 **Merged by:** [@michelroegl-brunner](https://github.com/michelroegl-brunner) **Base:** `new_script_testing` ← **Head:** `contribute` --- ### 📝 Commits (10+) - [`e8f0047`](https://github.com/community-scripts/ProxmoxVE/commit/e8f0047b2225257119dc546eca1634a6455b870b) First Design: CONTRIBUTING - [`7a7b0fa`](https://github.com/community-scripts/ProxmoxVE/commit/7a7b0facc1faa68c8665f52b4353e187546d7c8c) Update CONTRIBUTING.md - [`9ea997f`](https://github.com/community-scripts/ProxmoxVE/commit/9ea997f18b30f75be2f96d8a5cfc29e870bddb54) Update CONTRIBUTING.md - [`482bd08`](https://github.com/community-scripts/ProxmoxVE/commit/482bd088add63f4b674fca9d733cd986755124c9) Update CONTRIBUTING.md - [`cf7eb92`](https://github.com/community-scripts/ProxmoxVE/commit/cf7eb927ebe23d9b57980fc2b7f6f8f01c0bc87b) Update CONTRIBUTING.md - [`bb3077a`](https://github.com/community-scripts/ProxmoxVE/commit/bb3077a8c6835b595cf3be41ae8e2ed0117c1db9) Update CONTRIBUTING.md - [`d2728d2`](https://github.com/community-scripts/ProxmoxVE/commit/d2728d222d8bd1aed14d79cb19e4c53fbb273cc9) Update CONTRIBUTING.md - [`7094920`](https://github.com/community-scripts/ProxmoxVE/commit/70949207c15569e8281cd96d6c250fe0db9c558c) Update CONTRIBUTING.md - [`6a0f22c`](https://github.com/community-scripts/ProxmoxVE/commit/6a0f22c43067451de335f2f77addd06d6070e887) Update CONTRIBUTING.md - [`483dc98`](https://github.com/community-scripts/ProxmoxVE/commit/483dc981f3dd6582c237a8f018dc388c214b6f66) Update CONTRIBUTING.md ### 📊 Changes **12 files changed** (+1057 additions, -14 deletions) <details> <summary>View changed files</summary> ➖ `.github/CONTRIBUTING.md` (+0 -14) ➕ `.github/CONTRIBUTOR_GUIDE/CONTRIBUTING.md` (+108 -0) ➕ `.github/CONTRIBUTOR_GUIDE/ct/AppName.md` (+231 -0) ➕ `.github/CONTRIBUTOR_GUIDE/ct/AppName.sh` (+93 -0) ➕ `.github/CONTRIBUTOR_GUIDE/dev-scripts/README.md` (+42 -0) ➕ `.github/CONTRIBUTOR_GUIDE/dev-scripts/change_lxc.sh` (+52 -0) ➕ `.github/CONTRIBUTOR_GUIDE/dev-scripts/cleanup_lxc.sh` (+60 -0) ➕ `.github/CONTRIBUTOR_GUIDE/dev-scripts/new_lxc.sh` (+84 -0) ➕ `.github/CONTRIBUTOR_GUIDE/install/AppName-install.md` (+262 -0) ➕ `.github/CONTRIBUTOR_GUIDE/install/AppName-install.sh` (+86 -0) ➕ `.github/CONTRIBUTOR_GUIDE/json/AppName.json` (+34 -0) ➕ `.github/CONTRIBUTOR_GUIDE/json/AppName.md` (+5 -0) </details> ### 📄 Description > **🛠️ Note:** > We are meticulous about merging code into the main branch, so please understand that pull requests not meeting the project's standards may be rejected. It's never personal! > 🎮 **Note for game-related scripts:** These have a lower likelihood of being merged. --- ## ✍️ Description Please see everything inside the **dev-scripts** folder! I have a readme file with the breakdown and steps needed for the scripts. Essentially what these scripts do is to setup a branch for creating a new helper script and automate a couple of things that may be forgotten like changing the build.func, install.func etc to point to the new branch instead of the official repo! @michelroegl-brunner there you go mate, they're rough around the edge since they were developed for personal use but the workflow works! Although change_lxc.sh and cleanup_lxc.sh need a bit of more testing, new_lxc.sh seems to work for my needs pretty good (tested with 3-4 new helper scripts that I created a PR for them!) - - - **_Please remove unneeded lines!_** - Related Issue: # (issue number, if applicable) - Related PR: #920 (if applicable) - Related Discussion: [https://github.com/community-scripts/ProxmoxVE/pull/920#issuecomment-2563582115](https://github.com/community-scripts/ProxmoxVE/pull/920#issuecomment-2563582115) and [https://github.com/community-scripts/ProxmoxVE/pull/920#issuecomment-2563602712](https://github.com/community-scripts/ProxmoxVE/pull/920#issuecomment-2563602712)(if applicable) --- ## 🛠️ Type of Change Please check the relevant options: - [ ] Bug fix (non-breaking change that resolves an issue) - [ ] New feature (non-breaking change that adds functionality) - [ ] Breaking change (fix or feature that would cause existing functionality to change unexpectedly) - [ ] New script (a fully functional and thoroughly tested script or set of scripts) --- ## ✅ Prerequisites The following steps must be completed for the pull request to be considered: - [ ] Self-review performed (I have reviewed my code to ensure it follows established patterns and conventions.) - [ ] Testing performed (I have thoroughly tested my changes and verified expected functionality.) - [x] Documentation updated (I have updated any relevant documentation) --- ## 📋 Additional Information (optional) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-26 13:32:00 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/ProxmoxVE#2979
No description provided.