YunoHost-Apps / searx_ynh

Searx package for YunoHost
https://searx.github.io/searx/
GNU General Public License v3.0
20 stars 17 forks source link

Fix upgrade process, stretch support and linter #34

Closed Jibec closed 6 years ago

Jibec commented 6 years ago

Problems

Solution

PR Status

Could be reviewed and tested. It worked in x86 both package_check and VM. Needed: arm tests.

Validation


Minor decision

maniackcrudelis commented 6 years ago

Could you, please, rework this PR to start from the branch "add_ynh_check_starting", to avoid all this conflict and have a clear view about what's new in this PR.

And also explain what did you do.

maniackcrudelis commented 6 years ago

This PR does not seem to fix the issue in https://github.com/YunoHost-Apps/searx_ynh/pull/33 Can we have any explanation about this PR @Jibec ?

maniackcrudelis commented 6 years ago

And as I already said, you need to restart this PR from the correct branch. So please, close this PR, and redo the needed commits from the correct branch.

Also, this PR does not fix the issue in #33. So again, did you reproduce the error ?

I think that the fixes for the linter and missing packages should be done in another PR, in order to be merged separately. Since it seems legit.

PS: Avoid branch name like "origin/add_ynh_check_starting" which is difficult to handle for git cli and for our CI.