Hi all, I also think that abbreviations should not be mandatory (point 3). But what I am worrying about is a conflict resolution between a patch submitter and a reviewer. How to come to an agreement when one side is strictly for and another side is strictly against using abbreviations in some concrete case?
вс, 21 окт. 2018 г. в 11:34, Dmitriy Pavlov <dpavlov....@gmail.com>: > +1 for proposal 3. > > 1. I'm not sure we need to revisit all abbreviations as a lot of people get > used to it. > 2. I'm not sure multiword is always need to be fully named, sometimes it > may be ok to abbreviate. > 3. But I agree with abbreviations should not be mandatory. > > Abbreviated and short names like i,j,cp and etc. are good for simple > methods and code blocks; for a fast demonstration of some idea, but for > complex enterprise level software it can hide meaning instead of clearly > showing it. > > As a next step, I would like to propose to contribute an option to disable > abbreviation requirements for some cases in ignite-abbrev-plugin. > > сб, 20 окт. 2018 г. в 10:47, Zhenya <arzamas...@mail.ru.invalid>: > > > +1 for all proposals. > > > -- Best regards, Ivan Pavlukhin
