-
Notifications
You must be signed in to change notification settings - Fork 39
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
Global upgrade #43
Global upgrade #43
Conversation
Global upgrade
We should probably wait for #46, what do you think? |
Agree |
Alright I got tired of people reporting issues with this app so I fixed the regenconf hook There's a CI test pending here https://ci-apps-dev.yunohost.org/jenkins/view/aleks/job/my_webapp_ynh%20(aleks)/ I intend to yolomerge this PR if it passes tests |
Agreed to merge during today's meeting ... not sure why CI HQ fails but tests are passing on the dev CI ... (though at some point there was the same error). Will see |
Problem
Solution
PR Status
Validation
Minor decision
When the PR is marked as ready to merge, you have to wait for 3 days before really merging it.