search

YunoHost-Apps / dokuwiki_ynh

Dokuwiki package for YunoHost
https://www.dokuwiki.org/dokuwiki
GNU General Public License v3.0
11 stars 15 forks source link

Testing #112

Closed Gofannon closed 3 months ago

Gofannon commented 3 months ago

Problem

Solution

PR Status

Automatic tests

Automatic tests can be triggered on https://ci-apps-dev.yunohost.org/ after creating the PR, by commenting "!testme", "!gogogadgetoci" or "By the power of systemd, I invoke The Great App CI to test this Pull Request!". (N.B. : for this to work you need to be a member of the Yunohost-Apps organization)

Gofannon commented 3 months ago

!testme

yunohost-bot commented 3 months ago

:v: Test Badge

Gofannon commented 3 months ago

@ericgaspar , can you check this PR please? (I have no idea how this stuff is handled anymore so I'm tagging you directly)

Gofannon commented 3 months ago

have you tested it by hand?

No. I don't anymore. Because of theses reasons:

  1. time is missing
  2. changes in this PR are minors
  3. I used to test the 'testing' branch on my production server but:
    1. as there was "multiple iterations" applied to my server before getting a "stable version", the "test" was not really relevant anymore - multiple upgrades chained can lead to problems unseen.
    2. "testing" branch is not really used from my understanding (not tested by other people than me and communicating on the forum about it was time consuming )
    3. After my test, I was "stuck" on my "test branch" and YunoHost did not use the "normal version" from this repository and I did not realize it. I had to do a yunohost app upgrade dokuwiki --force to realign the package on my server with this repository.
OniriCorpe commented 3 months ago

so merging?

Gofannon commented 3 months ago

so merging?

Yes. Done, thanks !