search

cmsdaq / DAQAggregator

Aggregate monitoring data from the CMS DAQ system
0 stars 3 forks source link

One naming convention in snapshot serializer: camel case #45

Open gladky opened 7 years ago

gladky commented 7 years ago

I suggest we stick to one naming convention in serializers: camel case. Here is part of snapshot requested in Expert API with block_retri key.

"globalTtsStates": { "block_retri": { "state": "R_8", "percentWarning": 0, "percentBusy": 0 }, }

ghost commented 7 years ago

Will take care of this, thanks.

hsakulin commented 7 years ago

Hi, when switching to camel case, please make sure to keep acronyms such as TTS in all caps (or all lowercase if at the start). Hannes.

From: Maciej Gladki [notifications@github.com] Sent: 25 October 2016 08:57 To: cmsdaq/DAQAggregator Subject: [cmsdaq/DAQAggregator] One naming convention in snapshot serializer: camel case (#45)

I suggest we stick to one naming convention in serializers: camel case. Here is part of snapshot requested in Expert API with block_retri key.

"globalTtsStates": { "block_retri": { "state": "R_8", "percentWarning": 0, "percentBusy": 0 }, }

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://github.com/cmsdaq/DAQAggregator/issues/45, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ASanSvDL02QyitDgEgAfzPDSff693iU4ks5q3ahfgaJpZM4Kfn9W.