Skip to content

Conversation

@subsymbolic
Copy link
Contributor

Task/Issue URL: N/A

Description:
The kotlin formatter has changed a number of times over the course of this project. This PR brings the project's formatting in line with the latest kotlin formatting style.

Steps to test this PR:

  1. Run the formatter over the entire project except the cpp/third-party folder
  2. Check to see that files stay the same (other than exception mentioned in the notes below)

NOTES: The xml resource licenses get updated in a way that isn't an improvement, so I left these alone. It would be great to update our formatter to NOT make that update in the future.


Internal references:

Software Engineering Expectations
Technical Design Template

@brindy brindy self-assigned this Sep 10, 2018
@subsymbolic subsymbolic requested a review from brindy September 10, 2018 11:09
Copy link
Contributor

@brindy brindy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@subsymbolic subsymbolic merged commit 8cc63c5 into develop Sep 10, 2018
@subsymbolic subsymbolic deleted the feature/code_formatting branch September 10, 2018 12:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants