stevedlawrence commented on PR #919: URL: https://github.com/apache/daffodil/pull/919#issuecomment-1404445163
> I think it would be a good idea to run scalafmt after this refactoring with a very limited .scalafmt.conf which only sorts the import lines into the correct order. Yeah. After this the rename is merged would be a good time to configure and run everything through scalafmt and get all the churn done at once. Another thing we may want to do is add any rename and scalafmt commits to `.git-blame-ignore-revs` file. GitHub will use his in the blame view and local clones can be configured to use it with the blame.ignoreRevsFile. We would have to do it in a separate PR since merging PR's changes the hash, but we can do it once all the churn is done. -- 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]
