netudima commented on code in PR #4038: URL: https://github.com/apache/cassandra/pull/4038#discussion_r2281803203
########## .gitignore: ########## @@ -57,6 +57,9 @@ ide/nbproject/private nb-configuration.xml nbactions.xml +#VS code Review Comment: While technically I agree that this is non-related change I also think that a separate ticket/review and merge across branches for this very simple non-code change looks like an overkill here. My personal opinion: we need a balance between strict rules and usability to make the contribution process comfortable enough, so I personally prefer to focus on functional/performance and maintenance issues and trying to dig too much into style/taste comments.. -- 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: pr-unsubscr...@cassandra.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: pr-unsubscr...@cassandra.apache.org For additional commands, e-mail: pr-h...@cassandra.apache.org
