[GitHub] [ofbiz-framework] mbrohl commented on pull request #498: Improved: WorkEffort - MainActionMenu (OFBIZ-12557)

[GitBox <gi...@apache.org>]

mbrohl commented on pull request #498:
URL: https://github.com/apache/ofbiz-framework/pull/498#issuecomment-1029313372


   As said before, the removal of blank lines between sections makes the files harder to read.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@ofbiz.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [GitHub] [ofbiz-framework] mbrohl commented on pull request #498: Improved: WorkEffort - MainActionMenu (OFBIZ-12557)

[Pierre Smits <pi...@apache.org>]

Indeed, Michael. That was what you said  in comments regarding:

   1. https://github.com/apache/ofbiz-framework/pull/481
   2. https://github.com/apache/ofbiz-framework/pull/482
   3. https://github.com/apache/ofbiz-framework/pull/483

and now again in https://github.com/apache/ofbiz-framework/pull/498

What is this RULE of yours that fellow volunteering contributors MUST
comply with in order to have their improvements go into the codebase:

Is


it


single


line


spacing,


*or*


Is


ig


double


line


spacing?


It must surely feel very frustrating and annoying (even maybe painful?)
when YOU look at code in the OFBiz repositories and the majority of that
code not having the single or double line spacing that you want.
How unfortunate that all those contributors of code improvements over the
past 15 years of this project's existence (including you, contributing for
as long as that, as your claimed elsewhere) violated that RULE of yours
(that you brought forward just 11 days ago).

If the code of OFBiz (and improvements thereon) give you so much
difficulties to read, maybe there are some helpful tools available for you
to continue to bring your valuable contributions to improve this project
and its works? Or maybe it is time to call it quits?


Met vriendelijke groet,

Pierre Smits
*Proud* *contributor** of* Apache OFBiz <https://ofbiz.apache.org/> since
2008 (without privileges)
Proud contributor to the ASF since 2006
*Apache Directory <https://directory.apache.org>, PMC Member*

Anyone could have been you, whereas I've always been anyone.


On Thu, Feb 3, 2022 at 8:12 PM GitBox <gi...@apache.org> wrote:

>
> mbrohl commented on pull request #498:
> URL:
> https://github.com/apache/ofbiz-framework/pull/498#issuecomment-1029313372
>
>
>    As said before, the removal of blank lines between sections makes the
> files harder to read.
>
>
> --
> This is an automated message from the Apache Git Service.
> To respond to the message, please log on to GitHub and use the
> URL above to go to the specific comment.
>
> To unsubscribe, e-mail: notifications-unsubscribe@ofbiz.apache.org
>
> For queries about this service, please contact Infrastructure at:
> users@infra.apache.org
>
>
>