[GitHub] [incubator-pekko] mdedetrich commented on pull request #341: Add unwrap single string into .git-ignore-blame-revs

["mdedetrich (via GitHub)" <gi...@apache.org>]

mdedetrich commented on PR #341:
URL: https://github.com/apache/incubator-pekko/pull/341#issuecomment-1562357895

   > I don't think this should be done. I can agree to using this file for formatting only changes but I don't think code refactors like this should not be treated like this.
   
   This is a formatting only change, it has zero impact on any business logic or code. I was quite meticulous in doing/checming this.
   
   Unwrapping single string definitions isn't any different to lets say removing redundant parens which scalafmt does and hence the massive scafmt which we did at the start of the fork has these kinds of changes as well.
   
   There are also other commits in `.git-ignore-blame-revs` with similar reasoning.
   
   


-- 
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@pekko.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@pekko.apache.org
For additional commands, e-mail: notifications-help@pekko.apache.org