make getGaudiType a classmethod

@clemenci unwip as you please

@sstahl fyi

for the configuration framework, it would be nice to be able to get the gaudi type without having to instantiate.
needed for additional checks.

changed the description

mentioned in commit lhcb/Moore@e9e66879

mentioned in merge request lhcb/Moore!187 (merged)

The convention is to use cls for the first parameter of classmethods. In this case all these can be made @staticmethod https://docs.python.org/3/library/functions.html#staticmethod as you don't even need the class

added 1 commit

• 5b43f048 - make static methods of getGaudiType()

Compare with previous version

added 1 commit

• 1cc7f640 - add classmethod for gauditype

Compare with previous version

resolved all discussions

mentioned in commit lhcb/Moore@d0985521

mentioned in commit lhcb/Moore@2fe30057

FWIW, I would classify this as a bug fix

mentioned in commit lhcb/Moore@d24d80d4

mentioned in merge request lhcb/Moore!179 (closed)

mentioned in commit lhcb/Moore@84f579d1

mentioned in merge request lhcb/Moore!189 (merged)

added bug fix label

unmarked as a Work In Progress

assigned to @clemenci

changed milestone to %v32r1

• [2019-06-20 17:56] Validation started with lhcb-gaudi-merge#779
• [2019-06-21 00:06] Validation started with lhcb-lcg-dev3#932
• [2019-06-21 00:07] Validation started with lhcb-test-throughput2#64
• [2019-06-21 00:08] Validation started with lhcb-gaudi-head#2292
• [2019-06-21 00:08] Validation started with lhcb-lcg-dev4#940
• [2019-06-21 00:08] Validation started with lhcb-dd4hep#219
• [2019-06-21 00:09] Validation started with lhcb-soa-track#115
• [2019-06-21 00:12] Validation started with lhcb-tdr-test#584

approved this merge request