next-exp / nexus

Geant4 simulation framework of the NEXT Collaboration
5 stars 55 forks source link

Cosmetics #232

Closed paolafer closed 7 months ago

paolafer commented 7 months ago

This PR updates part of the code following our code conventions and improving its logic consistence.

paolafer commented 7 months ago

The length of each line of code should be at most 80-100 characters.

It seems that here you apply a strict 80-character limit. If that's the criterion to be followed I suggest updating the document. In my opinion, slightly longer lines read better if they avoid an otherwise unnecessary line break.

You're right, I got trapped in the red-pen syndrome 😅. Can you suggest some criteria or give me some examples of lines that would read better?

paolafer commented 7 months ago

Suggestions implemented!