Skip to content

Conversation

@dpursehouse
Copy link

No description provided.

Change-Id: I50dc9e8c4dbe8f44abcff169baabd13a4f861166
Signed-off-by: David Pursehouse <dpursehouse@collab.net>
Copy link
Author

@dpursehouse dpursehouse left a comment

Choose a reason for hiding this comment

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

@cushon any chance to get this merged? it'll need a new version of the Eclipse plugin to be published unless that update in the README is backed out of this commit.


A
[google-java-format Eclipse plugin](https://github.com/google/google-java-format/releases/download/google-java-format-1.6/google-java-format-eclipse-plugin_1.6.0.jar)
[google-java-format Eclipse plugin](https://github.com/google/google-java-format/releases/download/google-java-format-1.7/google-java-format-eclipse-plugin_1.7.0.jar)
Copy link
Author

Choose a reason for hiding this comment

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

Version 1.7 of the plugin wasn't published yet. See #331 .

@dpursehouse dpursehouse mentioned this pull request Apr 15, 2019
ronshapiro pushed a commit that referenced this pull request Apr 16, 2019
Fixes #343,#352,#350

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=243715974
@ronshapiro ronshapiro mentioned this pull request Apr 16, 2019
ronshapiro pushed a commit that referenced this pull request Apr 16, 2019
Fixes #343,#352,#350

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=243715974
@eaftan
Copy link
Contributor

eaftan commented Apr 16, 2019

I took one of the other 3 PRs that updated the README to 1.7. I didn't take the update to the Eclipse plugin because that needs followup on our part.

@eaftan eaftan closed this Apr 16, 2019
@dpursehouse dpursehouse deleted the fix-versions-in-README branch April 18, 2019 02:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants