Skip to content

Conversation

@kevinkowa
Copy link
Contributor

This PR contains the changes for the v8.4.1 release of the Java SDK with a patch to fix PassagesPerDocument for discovery-v2

The tags used for this release are:

git checkout sdk-2020-08-20
Generator: 2.3.1

mediumTaj
mediumTaj previously approved these changes Sep 24, 2020
Copy link
Contributor

@mediumTaj mediumTaj left a comment

Choose a reason for hiding this comment

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

👍 looks good!

mamoonraja
mamoonraja previously approved these changes Sep 24, 2020
Copy link
Contributor

@germanattanasio germanattanasio left a comment

Choose a reason for hiding this comment

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

LGTM

@kevinkowa kevinkowa requested a review from mediumTaj September 28, 2020 20:13
mediumTaj
mediumTaj previously approved these changes Sep 28, 2020
Copy link
Contributor

@mediumTaj mediumTaj left a comment

Choose a reason for hiding this comment

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

👍 looks good! One question but not a big deal....

super.setUp();
String apiKey = getProperty("discovery.apikey");
String url = getProperty("discovery.url");
String url = "https://api.us-south.discovery.watson.cloud.ibm.com";
Copy link
Contributor

Choose a reason for hiding this comment

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

Should this be hardcoded? Eventually we should update the credentials to use the proper urls.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

updated! b66e5d4

Copy link
Contributor

@mediumTaj mediumTaj left a comment

Choose a reason for hiding this comment

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

👍 looks good!

@kevinkowa kevinkowa merged commit 12679ae into master Sep 28, 2020
@kevinkowa kevinkowa deleted the sdk-passage-patch branch September 28, 2020 20:48
@watson-github-bot
Copy link
Contributor

🎉 This PR is included in version 8.6.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

6 participants