-
-
Notifications
You must be signed in to change notification settings - Fork 133
new: miniflux #1029
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
new: miniflux #1029
Conversation
|
Oh, now I see #935 🤦 |
|
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 |
|
@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 |
|
@phof we always accept both, debian and alpine. so if you wish you can reopen your PR anytime. |
|
@omernaveedxyz @michelroegl-brunner Reopened. I'll review the changes to the merged Debian Miniflux recipe and update this one; the update scripts likely needs improvement. |
82e8daf to
acc439a
Compare
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)
🛠️ Type of Change (X in brackets)
README,AppName.md,CONTRIBUTING.md, or other docs.🔍 Code & Security Review (X in brackets)
Code_Audit.md&CONTRIBUTING.mdguidelinesAppName.sh,AppName-install.sh,AppName.json)📋 Additional Information (optional)