tadayosi commented on PR #3464:
URL: https://github.com/apache/camel-k/pull/3464#issuecomment-1190158611

   > Unless we have an automated way of dealing with it, I'd prefer not to have 
any validation for the line length then. The risk is that most of the pull 
request will be invalid because we don't have a common and automatic way to 
format it.
   
   I can understand. At the same time, my motivation was that unless we set 
some limitation on the line length we'd easily extend code horizontally and 
there would be no opportunity to rethink about better design of function 
signatures. It seems to me the current code has gone a bit extreme as there are 
quite a few places where the lengths of lines go beyond 200.
   
   I thought that the default line length 120 imposed by `lll` is somewhat a 
common standard and good to follow.
   
   This time, I'll revert the `lll` part and only keep the golangci-lint 
upgrade before moving on.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to