Re: KDE review request: KDominate

Albert Astals Cid <[email protected]> Mon, 06 Apr 2026 16:13:53 +0200
Newsgroups gmane.comp.kde.devel.core
Message-ID <13766331.UUUT4zGiXx@xps15>
El dijous, 26 de mar=C3=A7 del 2026, a les 17:31:55 (Hora d=E2=80=99estiu d=
=E2=80=99Europa=20
central), Albert Vaca Cintora va escriure:
> The following checkboxes still need ticking :)
>=20
> - A screenshot in product-screenshots
> - A bugs.kde.org product (file a sysadmin ticket)
>=20
> These two are done (but I think I shouldn't tick the boxes myself, right?=
).

=46or the ones that are factual, i think you can. Either there is or there =
is=20
not a screenshot/product. I have done it for you anyway.

>=20
> - Check the code with some sanity tools like clazy or clang-tidy, if not
> already done as part of CI runs
>=20
> Not automated in the CI but I ran clazy a few times during development and
> fixed the findings.

I also checked this one.

>=20
> - Passing KDE neon build
>=20
> No idea what needs to be done here.
>=20
> https://invent.kde.org/games/kdominate/-/issues/1

Everything is checked now.

Cheers,
  Albert


>=20
> On Sun, Mar 1, 2026 at 2:31=E2=80=AFAM Albert Vaca Cintora <albertvaka@gm=
ail.com>
>=20
> wrote:
> > The repo has now been moved to https://invent.kde.org/games/kdominate so
> > the review can proceed.
> >=20
> > On Fri, Feb 27, 2026 at 9:20=E2=80=AFPM Albert Vaca Cintora <albertvaka=
@gmail.com>
> >=20
> > wrote:
> >> Hi,
> >>=20
> >> I wrote KDominate, a simple desktop game that I would like to make part
> >> of KDE Games.
> >>=20
> >> I reused the UI from KJumpingCube to keep the same look and feel as ot=
her
> >> KDE Games, although I modernized the code a bit (KJumpingCube was made=
 in
> >> 1999!). The README explains both how to play and how the code is
> >> structured, so it might be useful to read.
> >>=20
> >> The gameplay is actually based on a shareware game for Windows 3.1 tha=
t I
> >> was nostalgic about, maybe someone here even played the original :)
> >>=20
> >> * Source: https://invent.kde.org/albertvaka/kdominate
> >> * Review checklist:
> >> https://invent.kde.org/albertvaka/kdominate/-/issues/1
> >>=20
> >> Albert