[PR #2599] [MERGED] [gh] Update Script Test Workflow #3757

Closed
opened 2025-11-20 06:06:34 -05:00 by saavagebueno · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/community-scripts/ProxmoxVE/pull/2599
Author: @michelroegl-brunner
Created: 2/24/2025
Status: Merged
Merged: 2/24/2025
Merged by: @michelroegl-brunner

Base: mainHead: script-test-workflow


📝 Commits (2)

📊 Changes

5 files changed (+115 additions, -115 deletions)

View changed files

📝 .github/workflows/script-test.yml (+18 -29)
📝 .github/workflows/scripts/app-test/pr-alpine-install.func (+9 -11)
📝 .github/workflows/scripts/app-test/pr-build.func (+18 -18)
📝 .github/workflows/scripts/app-test/pr-create-lxc.sh (+49 -34)
📝 .github/workflows/scripts/app-test/pr-install.func (+21 -23)

📄 Description

✍️ Description

This PR updates the Script-Test workflow to instead of using local version of the script files to curl from the main Repo.

Prerequisites

Before this PR can be reviewed, the following must be completed:

  • Self-review performed – Code follows established patterns and conventions.
  • Testing performed – Changes have been thoroughly tested and verified.

🛠️ Type of Change

Select all that apply:

  • 🐞 Bug fix – Resolves an issue without breaking functionality.

📋 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/2599 **Author:** [@michelroegl-brunner](https://github.com/michelroegl-brunner) **Created:** 2/24/2025 **Status:** ✅ Merged **Merged:** 2/24/2025 **Merged by:** [@michelroegl-brunner](https://github.com/michelroegl-brunner) **Base:** `main` ← **Head:** `script-test-workflow` --- ### 📝 Commits (2) - [`9d98a46`](https://github.com/community-scripts/ProxmoxVE/commit/9d98a466f4ddb00fec5c45e4a5a8cef63185edb9) Update Workflow - [`a55b775`](https://github.com/community-scripts/ProxmoxVE/commit/a55b77560eb3cd2e90199e425f1d07db84e8d8cb) Update Workflow ### 📊 Changes **5 files changed** (+115 additions, -115 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/script-test.yml` (+18 -29) 📝 `.github/workflows/scripts/app-test/pr-alpine-install.func` (+9 -11) 📝 `.github/workflows/scripts/app-test/pr-build.func` (+18 -18) 📝 `.github/workflows/scripts/app-test/pr-create-lxc.sh` (+49 -34) 📝 `.github/workflows/scripts/app-test/pr-install.func` (+21 -23) </details> ### 📄 Description ## ✍️ Description <!-- Provide a clear and concise description of your changes. --> This PR updates the Script-Test workflow to instead of using local version of the script files to curl from the main Repo. ## ✅ Prerequisites Before this PR can be reviewed, the following must be completed: - [X] **Self-review performed** – Code follows established patterns and conventions. - [X] **Testing performed** – Changes have been thoroughly tested and verified. ## 🛠️ Type of Change Select all that apply: - [X] 🐞 **Bug fix** – Resolves an issue without breaking functionality. ## 📋 Additional Information (optional) <!-- Provide extra context, screenshots, or references if needed. --> --- <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 06:06:34 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: SVI/ProxmoxVE#3757