Skip to content

Commit 6124759

Browse files
author
SDKAuto
committed
CodeGen from PR 16221 in Azure/azure-rest-api-specs
Merge 82731da5d6c0260c3d1be337247bbf0333ab1eda into f727c69fa6aa618d706adae3fe8c1222d08d7a4d
1 parent e11f9d1 commit 6124759

23 files changed

+170
-1375
lines changed

common/config/rush/pnpm-lock.yaml

Lines changed: 15 additions & 20 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
Lines changed: 7 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -1,26 +1,11 @@
1-
## Release History
1+
# Release History
2+
3+
## 2.0.0 (2021-12-14)
24

3-
### 2.0.0-beta.2 (Unreleased)
5+
The package of @azure/arm-notificationhubs is using our next generation design principles since version 2.0.0, which contains breaking changes.
46

5-
#### Features Added
7+
To understand the detail of the change, please refer to [Changelog](https://aka.ms/js-track2-changelog).
68

7-
#### Breaking Changes
9+
To migrate the existing applications to the latest version, please refer to [Migration Guide](https://aka.ms/js-track2-migration-guide).
810

9-
#### Bugs Fixed
10-
11-
#### Other Changes
12-
13-
### 2.0.0-beta.1 (2021-11-01)
14-
15-
This is the first preview for the new version of the `@azure/arm-notificationhubs` package that follows the new [guidelines for TypeScript SDKs](https://azure.github.io/azure-sdk/typescript_introduction.html) for Azure services.
16-
17-
While this package remains auto generated, the SDK generator itself has undergone changes to comply with the above guidelines in order to generate packages that are idiomatic to the JavaScript/TypeScript ecosystem and consistent with other packages for Azure services. For more on this, please see [State of the Azure SDK 2021](https://devblogs.microsoft.com/azure-sdk/state-of-the-azure-sdk-2021/).
18-
19-
Please note that this version has breaking changes, all of which were made after careful consideration during the authoring of the guidelines and user studies.
20-
21-
**Noteworthy changes and features**
22-
- Authentication: The packages `@azure/ms-rest-nodeauth` or `@azure/ms-rest-browserauth` are no longer supported. Use package [@azure/identity](https://www.npmjs.com/package/@azure/identity) instead. Select a credential from Azure Identity examples based on the authentication method of your choice.
23-
- Callbacks: Method overloads that used callbacks have been removed and the use of promises is encouraged instead.
24-
- List operations now return an iterable result that follows the `PagedAsyncIterableIterator` interface as opposed to the previous model where you had to make a new request using the link to the next page.
25-
- Long running operations i.e. the Lro related object returned by methods whose names started with `begin`, now uses `pollUntilDone` to check whether the request is finished, instead of `pollUntilFinished`. To get the final result, use the corresponding method that will have the suffix `AndWait`.
26-
- The SDK only supports ECMAScript 2015 (ES6) and beyond, all projects that referenced this SDK should be upgraded to use ES6.
11+
To learn more, please refer to our documentation [Quick Start](https://aka.ms/js-track2-quickstart).

sdk/notificationhubs/arm-notificationhubs/README.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,10 @@ const subscriptionId = "00000000-0000-0000-0000-000000000000";
5353
const client = new NotificationHubsManagementClient(new DefaultAzureCredential(), subscriptionId);
5454
```
5555

56+
57+
### JavaScript Bundle
58+
To use this client library in the browser, first you need to use a bundler. For details on how to do this, please refer to our [bundling documentation](https://aka.ms/AzureSDKBundling).
59+
5660
## Key concepts
5761

5862
### NotificationHubsManagementClient
Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
{
2-
"commit": "4a2c1e8f277dd11a0da89d56eab8ff1a922d3a69",
2+
"commit": "0c382afe997f6706666311e96656bab022d30526",
33
"readme": "specification/notificationhubs/resource-manager/readme.md",
4-
"autorest_command": "autorest --version=3.1.3 --typescript --modelerfour.lenient-model-deduplication --head-as-boolean=true --license-header=MICROSOFT_MIT_NO_VERSION --generate-test --typescript-sdks-folder=D:\\mydev\\azure-sdk-for-js ../azure-rest-api-specs/specification/notificationhubs/resource-manager/readme.md --use=@autorest/[email protected]beta.13",
4+
"autorest_command": "autorest --version=3.1.3 --typescript --modelerfour.lenient-model-deduplication --head-as-boolean=true --license-header=MICROSOFT_MIT_NO_VERSION --generate-test --typescript-sdks-folder=/home/vsts/work/1/s/azure-sdk-for-js ../azure-rest-api-specs/specification/notificationhubs/resource-manager/readme.md --use=@autorest/[email protected]alpha.16.20211130.1",
55
"repository_url": "https://github.com/Azure/azure-rest-api-specs.git",
6-
"use": "@autorest/[email protected]beta.13"
6+
"use": "@autorest/[email protected]alpha.16.20211130.1"
77
}

sdk/notificationhubs/arm-notificationhubs/package.json

Lines changed: 9 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -3,26 +3,18 @@
33
"sdk-type": "mgmt",
44
"author": "Microsoft Corporation",
55
"description": "A generated SDK for NotificationHubsManagementClient.",
6-
"version": "2.0.0-beta.2",
7-
"engines": {
8-
"node": ">=12.0.0"
9-
},
6+
"version": "2.0.0",
7+
"engines": { "node": ">=12.0.0" },
108
"dependencies": {
119
"@azure/core-lro": "^2.2.0",
1210
"@azure/abort-controller": "^1.0.0",
13-
"@azure/core-paging": "^1.1.1",
11+
"@azure/core-paging": "^1.2.0",
1412
"@azure/core-client": "^1.0.0",
1513
"@azure/core-auth": "^1.3.0",
1614
"@azure/core-rest-pipeline": "^1.1.0",
1715
"tslib": "^2.2.0"
1816
},
19-
"keywords": [
20-
"node",
21-
"azure",
22-
"typescript",
23-
"browser",
24-
"isomorphic"
25-
],
17+
"keywords": ["node", "azure", "typescript", "browser", "isomorphic"],
2618
"license": "MIT",
2719
"main": "./dist/index.js",
2820
"module": "./dist-esm/src/index.js",
@@ -38,6 +30,7 @@
3830
"rollup-plugin-sourcemaps": "^0.4.2",
3931
"typescript": "~4.2.0",
4032
"uglify-js": "^3.4.9",
33+
"rimraf": "^3.0.0",
4134
"@azure/identity": "^2.0.1",
4235
"@azure-tools/test-recorder": "^1.0.0",
4336
"mocha": "^7.1.1",
@@ -48,9 +41,7 @@
4841
"type": "git",
4942
"url": "https://github.com/Azure/azure-sdk-for-js.git"
5043
},
51-
"bugs": {
52-
"url": "https://github.com/Azure/azure-sdk-for-js/issues"
53-
},
44+
"bugs": { "url": "https://github.com/Azure/azure-sdk-for-js/issues" },
5445
"files": [
5546
"dist/**/*.js",
5647
"dist/**/*.js.map",
@@ -70,14 +61,14 @@
7061
"types/*"
7162
],
7263
"scripts": {
73-
"build": "tsc && rollup -c 2>&1 && npm run minify && mkdirp ./review && npm run extract-api",
64+
"build": "npm run clean && tsc && rollup -c 2>&1 && npm run minify && mkdirp ./review && npm run extract-api",
7465
"minify": "uglifyjs -c -m --comments --source-map \"content='./dist/index.js.map'\" -o ./dist/index.min.js ./dist/index.js",
7566
"prepack": "npm run build",
7667
"pack": "npm pack 2>&1",
7768
"extract-api": "api-extractor run --local",
7869
"lint": "echo skipped",
7970
"audit": "echo skipped",
80-
"clean": "echo skipped",
71+
"clean": "rimraf dist dist-browser dist-esm test-dist temp types *.tgz *.log",
8172
"build:node": "echo skipped",
8273
"build:browser": "echo skipped",
8374
"build:test": "echo skipped",
@@ -86,14 +77,13 @@
8677
"execute:samples": "echo skipped",
8778
"format": "echo skipped",
8879
"test": "npm run integration-test",
89-
"prebuild": "echo skipped",
9080
"test:node": "echo skipped",
9181
"test:browser": "echo skipped",
9282
"unit-test": "npm run unit-test:node && npm run unit-test:browser",
9383
"unit-test:node": "cross-env TEST_MODE=playback npm run integration-test:node",
9484
"unit-test:browser": "echo skipped",
9585
"integration-test": "npm run integration-test:node && npm run integration-test:browser",
96-
"integration-test:node": "mocha -r esm --require ts-node/register --timeout 1200000 --full-trace test/*.ts",
86+
"integration-test:node": "mocha -r esm --require ts-node/register --timeout 1200000 --full-trace test/*.ts --reporter ../../../common/tools/mocha-multi-reporter.js",
9787
"integration-test:browser": "echo skipped",
9888
"docs": "echo skipped"
9989
},

0 commit comments

Comments
 (0)