From 7d782c26582d032f816e3d51eac1d24aa66f9bb4 Mon Sep 17 00:00:00 2001 From: Krakinou Date: Sun, 9 May 2021 14:04:51 +0200 Subject: [PATCH] Fix linter warnings --- check_process | 2 +- pull_request_template.md | 2 +- scripts/remove | 4 ++-- scripts/restore | 3 +-- scripts/upgrade | 2 +- 5 files changed, 6 insertions(+), 7 deletions(-) diff --git a/check_process b/check_process index 900985d..b06a4dd 100644 --- a/check_process +++ b/check_process @@ -19,7 +19,7 @@ backup_restore=1 multi_instance=0 port_already_use=0 #No port required - change_url=1 + change_url=0 #not yet done ;;; Levels # If the level 5 (Package linter) is forced to 1. Please add justifications here. Level 5=auto diff --git a/pull_request_template.md b/pull_request_template.md index 0287f9f..a747614 100644 --- a/pull_request_template.md +++ b/pull_request_template.md @@ -15,4 +15,4 @@ --- *If you have access to [App Continuous Integration for packagers](https://yunohost.org/#/packaging_apps_ci) you can provide a link to the package_check results like below, replacing '-NUM-' in this link by the PR number and USERNAME by your username on the ci-apps-dev. Or you provide a screenshot or a pastebin of the results* -[![Build Status](https://ci-apps-dev.yunohost.org/jenkins/job/REPLACEBYYOURAPP_ynh%20PR-NUM-%20(USERNAME)/badge/icon)](https://ci-apps-dev.yunohost.org/jenkins/job/REPLACEBYYOURAPP_ynh%20PR-NUM-%20(USERNAME)/) +[![Build Status](https://ci-apps-dev.yunohost.org/jenkins/job/retroarch_ynh%20PR-NUM-%20(USERNAME)/badge/icon)](https://ci-apps-dev.yunohost.org/jenkins/job/retroarch_ynh%20PR-NUM-%20(USERNAME)/) diff --git a/scripts/remove b/scripts/remove index 51fdd0a..97d611a 100755 --- a/scripts/remove +++ b/scripts/remove @@ -37,7 +37,7 @@ ynh_remove_app_dependencies # REMOVE CRONJOB #================================================= -ynh_secure_remove --file="/etc/cron.d/$app +ynh_secure_remove --file="/etc/cron.d/$app" #================================================= # REMOVE APP MAIN DIR @@ -68,4 +68,4 @@ fi # END OF SCRIPT #================================================= -ynh_script_progression --message="Removal of $app completed" --time --last +ynh_script_progression --message="Removal of $app completed" --time --last \ No newline at end of file diff --git a/scripts/restore b/scripts/restore index 2cf2ccb..4bf24d0 100755 --- a/scripts/restore +++ b/scripts/restore @@ -32,7 +32,6 @@ app=$YNH_APP_INSTANCE_NAME domain=$(ynh_app_setting_get --app=$app --key=domain) path_url=$(ynh_app_setting_get --app=$app --key=path) final_path=$(ynh_app_setting_get --app=$app --key=final_path) -is_public=$(ynh_app_setting_get --app=$app --key=is_public) #================================================= # CHECK IF THE APP CAN BE RESTORED @@ -85,7 +84,7 @@ ynh_script_progression --message="Reinstalling dependencies..." --time --weight= # Define and install dependencies #Dependencies are not really required as this is just to unzip the 7z file -#ynh_install_app_dependencies $pkg_dependencies +ynh_install_app_dependencies $pkg_dependencies #However, npm IS required to run the coffeescript ynh_install_nodejs --nodejs_version=10 ynh_use_nodejs diff --git a/scripts/upgrade b/scripts/upgrade index caf001a..a5828b0 100755 --- a/scripts/upgrade +++ b/scripts/upgrade @@ -103,7 +103,7 @@ ynh_script_progression --message="Upgrading dependencies..." --time --weight=1 # Define and install dependencies #Dependencies are not really required as this is just to unzip the 7z file -#ynh_install_app_dependencies $pkg_dependencies +ynh_install_app_dependencies $pkg_dependencies #However, npm IS required to run the coffeescript ynh_install_nodejs --nodejs_version=10 ynh_use_nodejs