[01:33:06]
<Yunohost Git/Infra notifications> [apps] @tituspijean merged [pull request #1732](https://github.com/YunoHost/apps/pull/1732): Update app levels according to CI results
[01:33:06]
<Yunohost Git/Infra notifications> [apps] @tituspijean pushed 5 commits to master ([93ce284015b6...2cc3ae6c4212](https://github.com/YunoHost/apps/compare/93ce284015b6...2cc3ae6c4212))
[01:33:08]
<Yunohost Git/Infra notifications> [apps] @tituspijean deleted branch update_app_levels
[01:33:12]
<Yunohost Git/Infra notifications> [apps/master] Update apps.toml - tituspijean
[01:33:12]
<Yunohost Git/Infra notifications> [apps/master] Update apps.toml - Éric Gaspar
[01:33:16]
<Yunohost Git/Infra notifications> [apps/master] Merge pull request #1732 from YunoHost/update_app_levels Update app levels according to CI results - tituspijean
[05:46:30]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @tomdereub [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/592#issuecomment-1722787514) on [issue #592](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/592) 26.0.6 (Oldstable): Hi, thanks a lot, is it already ready to upgrade ?
[08:50:47]
<Salamandar> > Oh nice, a new complex option to add in the generator 😅
In the "resources" section of the manifest, I would love to see a "languages" section, with node, python, go, rust available (
[08:51:00]
<Salamandar> (with the version)
[08:51:15]
<Salamandar> This would solve the "broken venvs on python major upgrades" issue.
[10:05:29]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar edited [pull request #596](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/596): V2
[10:06:15]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar edited [pull request #596](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/596): V2
[10:09:33]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar created new branch 27.1.0
[10:09:33]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar pushed 1 commit to 27.1.0: 27.1.0 ([68a471e1](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/68a471e16d1b8d5070c87de5d452eb9b8c61a716))
[10:09:34]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot pushed 1 commit to 27.1.0: Auto-update README ([17347df5](https://github.com/YunoHost-Apps/nextcloud_ynh/commit/17347df5299d0c74016dc26508a1706722cb694f))
[10:09:39]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar opened [pull request #604](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/604): 27.1.0
[10:10:12]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar edited [pull request #604](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/604): 27.1.0
[10:10:20]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @yunohost-bot [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/604#issuecomment-1723111757) on [issue #604](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/604) 27.1.0: :stuck_out_tongue_winking_eye:
[[Test Badge](https://img.shields.io/endpoint?url=https://ci-apps-dev.yunohost.org/ci/ap...
[10:10:20]
<Yunohost Git/Infra notifications> [nextcloud_ynh] @ericgaspar [commented](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/604#issuecomment-1723111710) on [issue #604](https://github.com/YunoHost-Apps/nextcloud_ynh/pull/604) 27.1.0: testme
[10:14:21]
<eric_G> I will be nice if we could move forward with this Nextcloud v2 PR(even though the upgrade from 25.0.2 is failing :( ): https://github.com/YunoHost-Apps/nextcloud_ynh/pull/596
[11:14:55]
<tituspijean> > This version of Nextcloud is not compatible with PHP>=8.2.<br/>You are currently running 8.2.10.+
Are you kidding me? 😅
[11:30:41]
<Aleks (he/him/il/lui)> hmmyeah maybe we could try discussin it during next meeting thursday ? I don't know if there are items to be careful about / source of worries ... but at least it feels like a big stuff and we need to be several person together to click the button haha
[11:30:56]
<Aleks (he/him/il/lui)> there's also the postgresql PR ?
[11:36:32]
<eric_G> > <@Alekswag:matrix.org> there's also the postgresql PR ?
Yes, (maintained because I use it on my production server 😬)
[11:37:04]
<Aleks (he/him/il/lui)> yeah we coud discuss it too
[11:37:23]
<Aleks (he/him/il/lui)> and have like idk a ceremony with candles and everything to click on merge
[11:37:30]
<Aleks (he/him/il/lui)> offerings for The Upgrade Gods
[11:37:54]
<Aleks (he/him/il/lui)> like let's buy bitcoins and then rm the private key idk
[11:38:10]
<eric_G> and maybe a little sacrifice.
[11:51:21]
<orhtej2> > <@ericg:matrix.org> I will be nice if we could move forward with this Nextcloud v2 PR(even though the upgrade from 25.0.2 is failing :( ): https://github.com/YunoHost-Apps/nextcloud_ynh/pull/596
If you think about it such update can be done by uninstall, install and `occ` command to rescan files, maybe we should just document it?
[11:51:57]
<orhtej2> (as by default data_dir is left untouched by uninstall)
[12:44:37]
<Daniel> I am reading the handbook on app packaging, I am kinda not very deep, but I have 2 initial questions:
1. I am familiar with bash, but managing complex bash scripts is hard for me, using bash as a thin wrapper for different runtime is okay or it is no-go
2. Can I get a recommendation for a small app that has "pristine" practices as a good example to learn from?
[12:49:34]
<Aleks (he/him/il/lui)> not sure what's question "1" 😅
[12:49:43]
<Aleks (he/him/il/lui)> as for "pristine" practice hmmm
[12:51:08]
<Aleks (he/him/il/lui)> kind of a random pick but let's say Hedgedoc : https://github.com/YunoHost-Apps/hedgedoc_ynh
[12:51:27]
<Aleks (he/him/il/lui)> level 8 (= that's the quality level derived from automatic tests) + it's using packaging v2
[12:51:40]
<Aleks (he/him/il/lui)> but you may be interested by other apps depending on technologies you will be working with
[12:52:49]
<Daniel> 1st is more like... when I have to maintain an app in future and I have to do more complicated migration which requires shuffling stuff around and migrating configs... and I think it would be hell to do that with seds - I would like to spin a quick deno/perl runtime for a migration
[12:55:05]
<Daniel> Okay, time to learn more and get hands dirty, maybe things will be clear when I will try it in practice :)
[12:56:17]
<Aleks (he/him/il/lui)> hmmm i'm not sure to get what you mean by "shuffling stuff around and migrating configs", i don't think that happens "that" much except for general packaging changes (eg packaging v1->v2 for which there is a tool that does 2/3rd of the work) ... but again i'm not sure what you're referring to
[12:56:50]
<Aleks (he/him/il/lui)> but if you have a complex migration to run and the migration ain't provided by the upstream then yeah, i guess you could write a script in whichever language that is easily maintainable by other folks, idk
[12:57:17]
<Aleks (he/him/il/lui)> hopefully upstreams don't yolo-change everything every other day
[12:57:21]
<Daniel> Everything's clear
[12:57:36]
<Daniel> Thank you ^^
[12:57:41]
<Aleks (he/him/il/lui)> 👍️
[13:08:52]
<Aleks (he/him/il/lui)> rodinux: i'm still heavily confused about the whole thing of backup/restore the entire data folder for paheko ... Yeswiki had a similar issue with `full_restore=1`+`keep` and ending up with some `data/data` structure, but at the end found a way to get it working
[13:55:05]
<Aleks (he/him/il/lui)> isAAAc: if you're interested about the yakform/framaform package, one missing stuff for packging v2 is to know if sso is working or not
[13:55:12]
<Aleks (he/him/il/lui)> but i'm kinda lazy to try it out
[13:55:28]
<Aleks (he/him/il/lui)> however I suspect there's no change with respect to the current version
[13:55:39]
<Aleks (he/him/il/lui)> do you know if SSO is working with the current version ?
[13:55:46]
<Aleks (he/him/il/lui)> (NB I'm talking about SSO, not LDAP)
[13:57:10]
<isAAAc> i don't know because the app is mainly broken with the actual package
[13:57:47]
<isAAAc> but i can try with the new one if it could be in testing branch
[13:58:02]
<isAAAc> or other solution
[13:59:03]
<isAAAc> oh, yes, even the apps is broken, framafroms is in the SSO and i can click on it
[14:00:34]
<Aleks (he/him/il/lui)> i mean do you get automatically logged-in without having to retype credentials ?
[14:00:44]
<Aleks (he/him/il/lui)> how is the app "broken" in the current state ..?
[14:14:06]
<isAAAc> > <@Alekswag:matrix.org> i mean do you get automatically logged-in without having to retype credentials ?
nope
[14:15:11]
<isAAAc> a module can't operate (i can't remember wich one, and I removed the app this night)
[14:15:27]
<Aleks (he/him/il/lui)> hmkay
[14:15:51]
<Aleks (he/him/il/lui)> in that case it could be nice if you can try to install the testing version of the app (after i merge the v2 PR) and test if you still reproduce the issue
[14:16:31]
<isAAAc> ok , i'll try that this evening, fire at work atm
[14:16:38]
<isAAAc> thx Aleks (he/him/il/lui)
[14:16:47]
<Aleks (he/him/il/lui)> alrighty
[14:19:39]
<Aleks (he/him/il/lui)> https://github.com/YunoHost-Apps/framaforms_ynh/pull/65#issuecomment-1723524075
[14:55:55]
<Yunohost Git/Infra notifications> [apps] @alexAubin edited [pull request #1717](https://github.com/YunoHost/apps/pull/1717): New app store
[14:56:56]
<Yunohost Git/Infra notifications> [apps] @alexAubin pushed 3 commits to app-store ([902b70618359...3ee0774acf1e](https://github.com/YunoHost/apps/compare/902b70618359...3ee0774acf1e))
[14:57:01]
<Yunohost Git/Infra notifications> [apps/app-store] appstore: redirect to the current page when login in using discourse - Alexandre Aubin
[14:57:03]
<Yunohost Git/Infra notifications> [apps/app-store] appstore: add usermenu with logout option - Alexandre Aubin
[14:57:08]
<Yunohost Git/Infra notifications> [apps/app-store] appstore: disable show only favorite when not loggedin - Alexandre Aubin
[16:50:08]
<Aleks (he/him/il/lui)> stan: the right way to install a virtual env is something along the lines of `python3 -m venv venv`, `source venv/bin/activate`, and `pip3 install -r requirements.txt`, but your issue probably lies somewhere else, devil is in the details, share the logs and code etc...
[16:53:06]
<stan> > <@Alekswag:matrix.org> stan: the right way to install a virtual env is something along the lines of `python3 -m venv venv`, `source venv/bin/activate`, and `pip3 install -r requirements.txt`, but your issue probably lies somewhere else, devil is in the details, share the logs and code etc...
I know how to install a virtual env, but I'm not able to find the requirements.txt from yunohost (the core) and I'm not able to launch pytest, even though I'm able to see the pytest.ini file
[16:53:38]
<Aleks (he/him/il/lui)> i'm confused ... what are you trying to achieve, are you talking about app packaging or testing yunohost's core ?
[16:53:52]
<stan> testing yunohost core
[16:54:03]
<stan> sorry if I was not clear
[16:54:11]
<Aleks (he/him/il/lui)> alrighty, then there is no virtualenv
[16:54:24]
<Aleks (he/him/il/lui)> we usually develop / tests in some LXC, though you can also use virtualbox or a test VPS
[16:54:37]
<Aleks (he/him/il/lui)> dependencies are installed via debian dependencies, because yunohost is a debian packag
[16:55:16]
<stan> so you say there is no unit tests ?
[16:55:23]
<Aleks (he/him/il/lui)> cf https://yunohost.org/dev for the general overview
[16:55:55]
<Aleks (he/him/il/lui)> we typically use `ynh-dev` https://github.com/yunohost/ynh-dev to setup a dev LXC easily
[16:56:28]
<Aleks (he/him/il/lui)> and then you can run tests locally using `./ynh-dev test yunohost` for example, though that will launch everything, you probably want to restrict to some module using eg `./ynh-dev test yunohost:apps`
[16:56:42]
<stan> oh ok
[16:56:47]
<stan> thanks
[16:57:33]
<Aleks (he/him/il/lui)> basically testing yunohost requires the entire ecosystem to be up, because it's just way too much to try to mock everything and you need like nginx, ldap, etc
[16:57:46]
<Aleks (he/him/il/lui)> so it's done directly inside some "actual" system, i.e. a test LXC
[16:57:56]
<Aleks (he/him/il/lui)> (or virtualbox or whatever)
[16:58:56]
<stan> ok, I understand, although I'm not used to it
[23:50:29]
<Yunohost Git/Infra notifications> [apps] @alexAubin pushed 8 commits to app-store ([3ee0774acf1e...662fcd2af3f0](https://github.com/YunoHost/apps/compare/3ee0774acf1e...662fcd2af3f0))
[23:50:43]
<Yunohost Git/Infra notifications> [apps/app-store] appstore: update translations - Alexandre Aubin
[23:50:47]
<Yunohost Git/Infra notifications> [apps/app-store] appstore: propagate changes from the current official wishlist - Alexandre Aubin
[23:50:48]
<Yunohost Git/Infra notifications> [apps] @alexAubin edited [pull request #1717](https://github.com/YunoHost/apps/pull/1717): New app store