Skip to content

Conversation

@cpovirk
Copy link
Member

@cpovirk cpovirk commented Apr 27, 2020

This code has been reviewed and submitted internally. Feel free to discuss on
the PR, and we can submit follow-up changes as necessary.

Commits:

Update the open-source google-java-format plugin for 2020.1.

I couldn't find any good way to make this backwards compatible. Oh well.

FYI, It looks like in 2020.1, we can probably actually use the
ExternalFormatProcessor extension point instead of doing all this hacky
nonsense... they added a #format(PsiFile, TextRange) method you can override.
But I'll leave that for later because 2020.1 is out now and people are mad.

a24b04f


Fix formatting of records without an explicit constructor

Fixes #460

a399e82

plumpy and others added 2 commits April 27, 2020 14:35
I couldn't find any good way to make this backwards compatible. Oh well.

FYI, It looks like in 2020.1, we can probably actually use the
ExternalFormatProcessor extension point instead of doing all this hacky
nonsense... they added a #format(PsiFile, TextRange) method you can override.
But I'll leave that for later because 2020.1 is out now and people are mad.

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=308647308
Fixes #460

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=308658958
@cpovirk cpovirk merged commit ca529a2 into master Apr 27, 2020
@cpovirk cpovirk deleted the sync-master-2020/04/27 branch April 27, 2020 18:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unable to format records

5 participants