update Elisa icons and add new release branch and set it as default
ClosedPublic

Authored by mgallien on Nov 18 2019, 9:40 PM.

Details

Summary

Elisa icon has changed, update it

add the new release branch in the branch list and switch it as default

Test Plan

none

Diff Detail

Repository
R877 Craft Blueprints for KDE
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
mgallien requested review of this revision.Nov 18 2019, 9:40 PM
mgallien created this revision.
vonreth added inline comments.
extragear/elisa/elisa.py
9

Looks good but the default target should not point to a branch. Please rather change the nightly build target in binaryfactory

mgallien updated this revision to Diff 70003.Nov 19 2019, 4:40 PM

set latest beta release version as default

@vonreth Thanks for the review

extragear/elisa/elisa.py
9

I replaced it by the last tagged release (v19.11.80)

vonreth added inline comments.Nov 20 2019, 12:48 PM
extragear/elisa/elisa.py
9

Any reason you need to set the default target to an unstable version?
This is what the stable build will produce.
Please modify https://invent.kde.org/sysadmin/binary-factory-tooling/blob/master/craft/enabled-projects.yaml#L59 instead.

mgallien updated this revision to Diff 70061.Nov 20 2019, 4:41 PM
  • switch default to the latest stable tag (0.4.2 version) for elisa

Sorry for not understanding what you meant. I have fixed the default target to the latest stable tag.

mgallien added inline comments.Nov 20 2019, 4:56 PM
extragear/elisa/elisa.py
9

I have now done exactly what you suggested. Thanks for your invaluable help once more.

vonreth accepted this revision.Nov 20 2019, 5:43 PM
vonreth added inline comments.
extragear/elisa/elisa.py
9

Great, I just realised that the versions.ini is basically not used as all targets are defined here.
Could you remove it and remove the call to

self.versionInfo.setDefaultValues()

Or go all for the versions.ini but I think the current setup is confusing.
Then just push :)

This revision is now accepted and ready to land.Nov 20 2019, 5:43 PM
mgallien updated this revision to Diff 70073.Nov 20 2019, 6:13 PM
  • switch default to the latest stable tag (0.4.2 version) for elisa
  • remove all versions specified in elisa.py, only use version.ini
This revision was automatically updated to reflect the committed changes.