Skip to content

GitLab

  • Menu
Projects Groups Snippets
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • N neutrinet_ynh
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 2
    • Issues 2
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 0
    • Merge requests 0
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Neutrinet
  • neutrinet_ynh
  • Merge requests
  • !31

Merged
Created May 03, 2020 by Aleks@aleksContributor0 of 3 tasks completed0/3 tasks

Misc cleaning to make tests and linter happier on the CI

  • Overview 16
  • Commits 20
  • Changes 10

Hey there !

I'm a bit late, but as promised, here's a PR (well, here it's called MR :D) to make the app pass tests on the CI.

I ended up cleaning a few things to adapt the app to common practices (maybe more things could be done but it feels good enough). Many things are made simpler because either a) Yunohost's core does the check / normalizing before running the script and b) Using Yunohost nginx's helpers to handle nginx conf.

The most touchy thing is the handling of the path/path_url variable ... that thing is an horrible mess and I internally screamed during the whole thing, but basically the setting should be called path but the bash variable should be called $path_url. If you don't do this things ain't gonna work. (Yes, (╯°□°)╯︵ ┻━┻)

I was in fact surprised to see the var in manifest called path_url and not path ... but I see that this got changed recently and the neutrinet cube script is still using path here : https://gitlab.domainepublic.net/Neutrinet/neutrinet_cube_install/-/blob/stable/neutrinet_cube_install.sh#L842 ... (and even in unstable?) so I'm guessing it's okay to brutally revert to path.

Remaining todo is :

  • Adding a LICENSE file to make linter happy. But since I'm only a one-time contributor to this package, I don't know what you wanna choose ?
  • I'm triggering tests on the dev CI in the background ... currently level 2 ... need to re-trigger a test after a typo fix for the upgrade : https://ci-apps-dev.yunohost.org/jenkins/view/aleks/job/neutrinet_ynh%20(aleks)/3/console (don't worry too much about the screaming red errors about template for now ... ) App is level 4, but will be 7 once a LICENSE file added 😉
  • Bump ~ynh version number in manifest ?
Edited Nov 30, 2020 by HgO
Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: misc_cleaning