Skip to content

Updated dependencies to make the build run on JDK9+#58

Merged
nwbirnie merged 2 commits intogoogleads:masterfrom
Raibaz:jdk-11-dependencies
Feb 19, 2019
Merged

Updated dependencies to make the build run on JDK9+#58
nwbirnie merged 2 commits intogoogleads:masterfrom
Raibaz:jdk-11-dependencies

Conversation

@Raibaz
Copy link
Copy Markdown
Contributor

@Raibaz Raibaz commented Feb 7, 2019

No description provided.

Copy link
Copy Markdown
Member

@jradcliff jradcliff left a comment

Choose a reason for hiding this comment

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

LGTM, but added Nick as a 2nd reviewer

Copy link
Copy Markdown
Contributor

@nwbirnie nwbirnie left a comment

Choose a reason for hiding this comment

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

Could you comment on what testing has been done please?

Comment thread pom.xml Outdated
Comment thread pom.xml Outdated
@Raibaz Raibaz force-pushed the jdk-11-dependencies branch from 7ba0f66 to fbeda1f Compare February 11, 2019 02:42
@Raibaz
Copy link
Copy Markdown
Contributor Author

Raibaz commented Feb 11, 2019

@nwbirnie, this has been tested building and running the authentication, basicoperations and reporting examples on JDK 1.8, 10.0.2 and 11 on OS X and on JDK 1.8 on Linux.

@nwbirnie nwbirnie merged commit dfd53b3 into googleads:master Feb 19, 2019
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.

3 participants