Skip to content
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

Regen trial v5 #978

Merged
merged 6 commits into from
Oct 4, 2019
Merged

Regen trial v5 #978

merged 6 commits into from
Oct 4, 2019

Conversation

dpopp07
Copy link
Contributor

@dpopp07 dpopp07 commented Oct 3, 2019

Full regeneration and reliance on the official release of the core. Getting this into the release branch before I finish the readme/migration and open a PR for the release.

The majority of the changes are for the Promises work.

@dpopp07 dpopp07 requested a review from mediumTaj October 3, 2019 21:57
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!

@mediumTaj
Copy link
Contributor

mediumTaj commented Oct 3, 2019

but should generate discovery using ef9010ab7de1b1fc7628a340cab118e2f00dd51d. Can do that in another PR though. There were some issues with the cpd only functionality that was added here.,

@codecov-io
Copy link

codecov-io commented Oct 4, 2019

Codecov Report

❗ No coverage uploaded for pull request base (release-v5-rc2@a0da761). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@               Coverage Diff               @@
##             release-v5-rc2   #978   +/-   ##
===============================================
  Coverage                  ?   100%           
===============================================
  Files                     ?      1           
  Lines                     ?      9           
  Branches                  ?      2           
===============================================
  Hits                      ?      9           
  Misses                    ?      0           
  Partials                  ?      0

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 a0da761...0ff0854. Read the comment docs.

* update the tests to match the style guide
@dpopp07 dpopp07 merged commit abb0bb4 into release-v5-rc2 Oct 4, 2019
@dpopp07 dpopp07 deleted the regen-trial-v5 branch October 4, 2019 04:37
dpopp07 added a commit that referenced this pull request Oct 4, 2019
* refactor: update authorization, stt, and tts to handle promises from the core

* refactor: all service methods now return a Promise by default - callbacks are still supported

* chore: support array parameters in visual recognition

* build: bump core version to 1.0.0

* docs: update the migration guide with final service updates

* chore: make `prefix` required in method `getAutocomplete`

* update the tests to match the style guide
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.

3 participants