Skip to content

V4.0.0 branch #854

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 159 commits into from
Mar 28, 2019
Merged

V4.0.0 branch #854

merged 159 commits into from
Mar 28, 2019

Conversation

dpopp07
Copy link
Contributor

@dpopp07 dpopp07 commented Mar 28, 2019

This branch contains all of the work that has been done for major version 4.

This PR is huge but the vast majority of this code has already been reviewed and approved on its way into the protected v4.0.0-branch.

There are also a lot of commits but many of them contain the "breaking change" messages that will make it into the changelog.

I welcome any questions or feedback regarding the process of moving this code into master

dpopp07 and others added 30 commits September 26, 2018 09:29
BREAKING CHANGE: Node 4 will no longer be supported, considered during development, or tested with.

To migrate your code, upgrade to a newer major version of Node.
## [3.11.1](v3.11.0...v3.11.1) (2018-09-28)

### Bug Fixes

* **icp:** disabling ssl verification now works for websocket connection ([f8466c8](f8466c8))
BREAKING CHANGE: Support for the `api_key` parameter has been removed.

For instances of Visual Recognition, use `iam_apikey` to authenticate.
# [3.12.0](v3.11.1...v3.12.0) (2018-10-10)

### Features

* **discovery:** new parameters added to match updates to the service ([838b044](838b044))
BREAKING CHANGE: The Dialog service will no longer be available

To migrate your code, use the Assistant v1 or v2 service.
dpopp07 added 22 commits March 21, 2019 11:02
BREAKING CHANGE: There is no more `index.ts` file, so importing with `require('watson-developer-cloud')` will no longer work.

To migrate your code, import only the services that you need, e.g. `require('watson-developer-cloud/speech-to-text/v1')`.

Using the file `sdk.ts` to export all of the services for browserification
…als`

update ibm-cloud-sdk-core, which renames `getCredentials` to `getServiceCredentials`

BREAKING CHANGE: These method renames are breaking changes. Update the method names to migrate your code.
BREAKING CHANGE: For `updateValue`, parameter `new_type` is changed to `new_value_type`. For `updateDialogNode`, parameter `new_type` is changed to `new_node_type`.
BREAKING CHANGE: Parameter `model_id` has been changed to `model` for the following methods: convertToHtml, classifyElements, extractTables, compareDocuments, deleteFeedback, getFeedback, createBatch, and updateBatch
BREAKING CHANGE: For `updateClassifier` and `createClassifier`, the parameter `{classname}_positive_examples` is changed to a map called `positive_examples` with classnames as keys.

See the migration guide, UPGRADE-4.0.md, for more information.
@dpopp07 dpopp07 requested a review from germanattanasio March 28, 2019 20:05
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

@codecov-io
Copy link

codecov-io commented Mar 28, 2019

Codecov Report

Merging #854 into master will decrease coverage by 6.1%.
The diff coverage is 81.28%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #854      +/-   ##
==========================================
- Coverage   95.65%   89.54%   -6.11%     
==========================================
  Files           2       26      +24     
  Lines          46     3519    +3473     
  Branches        2      303     +301     
==========================================
+ Hits           44     3151    +3107     
- Misses          2      357     +355     
- Partials        0       11      +11
Impacted Files Coverage Δ
assistant/v1.ts 92.56% <ø> (ø)
visual-recognition/v3.ts 80% <ø> (ø)
text-to-speech/v1-generated.ts 95.27% <ø> (ø)
speech-to-text/v1-generated.ts 93.71% <ø> (ø)
visual-recognition/v3-generated.ts 92.59% <ø> (ø)
natural-language-classifier/v1.ts 80% <ø> (ø)
discovery/v1-generated.ts 91.82% <ø> (ø)
tone-analyzer/v3.ts 80% <ø> (ø)
tone-analyzer/v3-generated.ts 96.05% <ø> (ø)
text-to-speech/v1.ts 42.85% <ø> (ø)
... and 34 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update be3ef04...bd74547. Read the comment docs.

@dpopp07 dpopp07 merged commit 08e9784 into master Mar 28, 2019
@dpopp07 dpopp07 deleted the v4.0.0-branch branch March 29, 2019 20:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants