You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I've noticed that some PRs are implementing one thing but then tidying up the formatting/adding license headers to seeming unrelated files (which is awesome) but I think we can slightly reduce the cognitive load of the reviewers by enforcing that these things are fixed (as much as possible) at the time of implementation. We can use Spotless to fix the formatting via ktlint and enforce license headers.
The text was updated successfully, but these errors were encountered:
I've noticed that some PRs are implementing one thing but then tidying up the formatting/adding license headers to seeming unrelated files (which is awesome) but I think we can slightly reduce the cognitive load of the reviewers by enforcing that these things are fixed (as much as possible) at the time of implementation. We can use Spotless to fix the formatting via ktlint and enforce license headers.
The text was updated successfully, but these errors were encountered: