[PR #1482] [MERGED] New script: SQL Server 2022 #3042

Open
opened 2025-11-20 05:33:09 -05:00 by saavagebueno · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/community-scripts/ProxmoxVE/pull/1482
Author: @kris701
Created: 1/14/2025
Status: Merged
Merged: 1/21/2025
Merged by: @MickLesk

Base: mainHead: newscript-sqlserver2022


📝 Commits (2)

📊 Changes

3 files changed (+149 additions, -0 deletions)

View changed files

ct/sqlserver2022.sh (+49 -0)
install/sqlserver2022-install.sh (+57 -0)
json/sqlserver2022.json (+43 -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

I have attempted to make a script to create a MS SQL Server 2022.
This is the first time i have tried to make a script for this project, so i understand if i did something wrong or if there is something i should change in it.
I have tested it on my own proxmox cluster, and it seems to work.


🛠️ 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)

Provide any extra context or screenshots about the feature or fix here.


🔄 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/1482 **Author:** [@kris701](https://github.com/kris701) **Created:** 1/14/2025 **Status:** ✅ Merged **Merged:** 1/21/2025 **Merged by:** [@MickLesk](https://github.com/MickLesk) **Base:** `main` ← **Head:** `newscript-sqlserver2022` --- ### 📝 Commits (2) - [`dcec260`](https://github.com/community-scripts/ProxmoxVE/commit/dcec2605a40821572b1704031e9f450494a8bfd0) Added script files - [`d3ed282`](https://github.com/community-scripts/ProxmoxVE/commit/d3ed282375f3cec717199beb6b1b924d6b40596e) Update sqlserver2022-install.sh ### 📊 Changes **3 files changed** (+149 additions, -0 deletions) <details> <summary>View changed files</summary> ➕ `ct/sqlserver2022.sh` (+49 -0) ➕ `install/sqlserver2022-install.sh` (+57 -0) ➕ `json/sqlserver2022.json` (+43 -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 I have attempted to make a script to create a MS SQL Server 2022. This is the first time i have tried to make a script for this project, so i understand if i did something wrong or if there is something i should change in it. I have tested it on my own proxmox cluster, and it seems to work. --- ## 🛠️ 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) - [x] 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: - [x] Self-review performed (I have reviewed my code to ensure it follows established patterns and conventions.) - [x] Testing performed (I have thoroughly tested my changes and verified expected functionality.) - [x] Documentation updated (I have updated any relevant documentation) --- ## 📋 Additional Information (optional) Provide any extra context or screenshots about the feature or fix here. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
saavagebueno added the pull-request label 2025-11-20 05:33:09 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: SVI/ProxmoxVE#3042