Skip to content

Conversation

@tremor021
Copy link
Member

✍️ Description

  • Updated AppName.md to include new format of default container vars
  • Updated AppName.sh to include new format of default container vars and moved comments on top of the line it refers to, as in the rest of the docs
  • Updated AppName-install.md to use curl -fsSL in code examples instead of wget
  • Updated AppName-install.sh with some code cleanup and formatting

🔗 Related PR / Issue

Link: #

✅ Prerequisites (X in brackets)

  • Self-review completed – Code follows project standards.
  • Tested thoroughly – Changes work as expected.
  • No security risks – No hardcoded secrets, unnecessary privilege escalations, or permission issues.

🛠️ Type of Change (X in brackets)

  • 🐞 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.

@tremor021 tremor021 self-assigned this May 15, 2025
@tremor021 tremor021 requested a review from a team as a code owner May 15, 2025 21:53
@github-actions github-actions bot added github maintenance Code maintenance or general upkeep of the project labels May 15, 2025
@MickLesk MickLesk merged commit ce69a57 into community-scripts:main May 16, 2025
1 check passed
@tremor021 tremor021 deleted the contrib branch May 18, 2025 11:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

github maintenance Code maintenance or general upkeep of the project

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants