-
Notifications
You must be signed in to change notification settings - Fork 5.6k
Update nodejs config for cognitive services #2948
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -6,7 +6,7 @@ Please also specify `--node-sdks-folder=<path to root folder of your azure-sdk-f | |
| ``` yaml $(nodejs) | ||
| nodejs: | ||
| package-name: azure-cognitiveservices-imagesearch | ||
| package-version: 1.0.0-preview | ||
| package-version: 1.0.0 | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Just curious, are these parts of Cognitive service live separately? So there is a package for VideoSearch and then another one for VisualSearch?
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yeah, we publish a different npm package for each one. |
||
| output-folder: $(node-sdks-folder)/lib/services/imageSearch | ||
| override-client-name: ImageSearchAPIClient | ||
| azure-arm: false | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,16 @@ | ||
| ## Node.js | ||
|
|
||
| These settings apply only when `--nodejs` is specified on the command line. | ||
| Please also specify `--node-sdks-folder=<path to root folder of your azure-sdk-for-node clone>`. | ||
|
|
||
| ``` yaml $(nodejs) | ||
| nodejs: | ||
| package-name: azure-cognitiveservices-visualsearch | ||
| package-version: 1.0.0-preview | ||
| output-folder: $(node-sdks-folder)/lib/services/visualSearch | ||
| override-client-name: VisualSearchAPIClient | ||
| azure-arm: false | ||
| generate-license-txt: true | ||
| generate-package-json: true | ||
| generate-readme-md: false | ||
| ``` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note that the version we've actually published is 1.1.0--not likely to publish a new one as it looks like swagger has not changed. Just doing a correction here