Skip to content

Conversation

@phof
Copy link

@phof phof commented Oct 23, 2025

Scripts wich are clearly AI generated and not further revied by the Author of this PR (in terms of Coding Standards and Script Layout) may be closed without review.

✍️ Description

New recipe for an Alpine-flavored Miniflux installation

🔗 Related PR / Issue

N/A

✅ Prerequisites (X in brackets)

  • Self-review completed – Code follows project standards.
  • Tested thoroughly – Changes work as expected.
  • No breaking changes – Existing functionality remains intact.
  • 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.

🔍 Code & Security Review (X in brackets)

  • Follows Code_Audit.md & CONTRIBUTING.md guidelines
  • Uses correct script structure (AppName.sh, AppName-install.sh, AppName.json)
  • No hardcoded credentials

📋 Additional Information (optional)

@phof phof marked this pull request as ready for review October 23, 2025 14:29
@phof phof requested review from a team as code owners October 23, 2025 14:29
@phof
Copy link
Author

phof commented Oct 23, 2025

Oh, now I see #935 🤦

@phof phof closed this Oct 23, 2025
@omernaveedxyz
Copy link

Hey, sorry, it looks like I got to it first 😁. However, I noticed you’ve created scripts for Alpine rather than Debian, so there may still be value in adding your changes on top of my PR with a few adjustments to match the formatting. Let me know what you think. Glad to see others using Miniflux

@phof
Copy link
Author

phof commented Oct 27, 2025

@omernaveedxyz Are we creating 2 recipes, one for debian and one for alpine? Or do you want to migrate to alpine? Happy to help.

@omernaveedxyz
Copy link

@omernaveedxyz Are we creating 2 recipes, one for debian and one for alpine? Or do you want to migrate to alpine? Happy to help.

I say we have both recipes and let the user choose what they want. I'm pretty sure there's a number of other recipes that offer both Debian and alpine

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants