Skip to content

Conversation

@CrazyWolf13
Copy link
Member

✍️ Description

Quickly fix the homarr app var to be lowercase, as the deployment path uses $app var, previously it was /opt/homarr and after update it would be changed to /opt/Homarr, brekaing installs even more.

🔗 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.

@CrazyWolf13 CrazyWolf13 self-assigned this May 14, 2025
@CrazyWolf13 CrazyWolf13 requested a review from a team as a code owner May 14, 2025 14:29
@CrazyWolf13 CrazyWolf13 added the invalid This doesn't seem right label May 14, 2025
@github-actions github-actions bot added bugfix update script A change that updates a script labels May 14, 2025
@MickLesk MickLesk merged commit c30f969 into main May 14, 2025
2 checks passed
@michelroegl-brunner michelroegl-brunner deleted the CrazyWolf13-patch-2 branch May 15, 2025 13:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix invalid This doesn't seem right update script A change that updates a script

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants