kde.org application pages: add link to CI per app on developer subpage
ClosedPublic

Authored by kossebau on Feb 12 2019, 9:05 PM.

Details

Summary

With the recently added feature of build.,kde.org to have repo-centric overview pages it might be interesting for one or the other to have the option to get directly from kde.org/applications/foo to the foo CI center.

Proposed additition to the app metadata schema is
"ci": [Product group, repo name]

Example given by Okteta metadata change, would extend to all other apps if okayed.

Test Plan

Untested

Diff Detail

Repository
R883 Subversion
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
kossebau requested review of this revision.Feb 12 2019, 9:05 PM
kossebau created this revision.

Needs D18974

@bcooksley Your input might be interesting as well, if the proposed metadata (product groupname, repo name) would be sufficient, or if you envision changes to build.kde.org which already should be considered here.

I can't forsee any additional changes at this stage in the structure - doing it by Product (roughly aligns to Release Units, with the exception of Extragear) then by Project (Repository) should be more than enough.

applications/applicationpage.php
224 ↗(On Diff #51547)

Perhaps we should use "View automated build (CI) results" or "View results of automated builds (CI)" as the wording here?

kossebau updated this revision to Diff 51591.Feb 13 2019, 2:33 PM
kossebau marked an inline comment as done.
kossebau added inline comments.
applications/applicationpage.php
224 ↗(On Diff #51547)

Agreeing to the native speaker :)

bcooksley accepted this revision.Feb 14 2019, 8:41 AM

Thanks, this is good to go now once the other dependent changes have gone in.

This revision is now accepted and ready to land.Feb 14 2019, 8:41 AM
This revision was automatically updated to reflect the committed changes.
kossebau marked an inline comment as done.

Seems to work, so going now to add the ci data to to some more json files (and then to bed, dreaming all of this could be fed from a central datastore ;) )

aacid added a comment.Feb 15 2019, 9:55 PM

Seems to work, so going now to add the ci data to to some more json files (and then to bed, dreaming all of this could be fed from a central datastore ;) )

Harald was working on this, but then stopped after a comment from my side that was a bit rough, Riddell said he may pick it up, but yes, the idea is there, and the code is almost there too