Increment version for eventhubs releases #1
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
After merging the recorder.skip changes, the nightly builds highlighted that our browser live tests are now failing. This is due to the fact that some of these tests are not expected to ever run on the browser, and recorder.skip will not work if TEST_MODE is live.
It is not recommended to map rolled-up files in the browser section. This change removes them. Also stop shipping browser bundles for several packages that we forgot to do so.
* [eslint-plugin] Moved our eslint plugin here - Configured template to use new plugin Archaeological note: eslint plugin previously located at https://github.com/Azure/azure-sdk-tools/ in /tools/eslint-plugin-azure-sdk Signed-off-by: Will Temple <witemple@microsoft.com> * Fixed definition of build step for eslint-plugin * Added definition for 'pack' and marked plugin private * Shimmed dom types for @azure/template * Template in shape * Rename .eslintrc.json .eslintrc.old.json * Replaced linting commands to reference .old.json - also fixed recorder linting configuration * Added unit-test command and fixed test command * Removed ci.yml, to add it back later with a correct configuration * Removed createDefaultProgram from linter's own linter settings. * Removed LICENSE file * Switched template to unified tsconfig * Set template version to 1.0.0 * Fixed some broken tests resulting from parser getting smarter
* [event-hubs] add security_token to management apis * add comments and changelog entry
as Azure devops pipeline only support one. Publishing for both NodeJs and browser would result in one of the report overwrites the other. Instead browser code coverage report are uploaded as an artifact.
* [storage][test] fetch next page until non-empty result array The service may return a segment with a valid continuation token, but with empty result array. In this case we need to keep getting next page until we have a non-empty result array. Remove the duplicate "BlobClient Node.js only undelete" test * more diagnostic logging * More logging * turn on info level logging * Add more asserts * empty result means array length of zero. * Another way of looping * Add one more condition to the loop exit check * set log level to info * log res * check deleteRetentionPolicy enabled * add assert on deleteRetentionPolicy.enabled * check deleteRetentionPolicy.enabled immediately * record test Co-authored-by: Lin Jian <ljian3377@gmail.com>
* Rework use of lib:dom in core-http Signed-off-by: Will Temple <witemple@microsoft.com> * rename lib/shims-internal.d.ts to lib/dom.d.ts * Removed shims from textanalytics * Renamed FetchInput to CommonRequestOptions for similarity to the DOM type * Added dom-shim.d.ts to package.json files * Added env flag to enable ts-node to lookup ambient declarations in dts files
* require core-http version of ^1.0.3 Increase minimum core-http version from 1.0.0 to 1.0.3 to incorporate important bug fixes. One data-loss bug fixed in 1.0.3 is #6656 * Update rest of storage libraries to depend on core-http 1.0.3 * Make core-http 1.0.3 the alternative/exception Co-authored-by: Jeremy Meng <yumeng@microsoft.com>
Log message and type correctly
* [core-lro] Using await to test that pollUntilDone throws * formatting
- double the time for large data tests to 20 minutes. We have seen them passing closer to 10-minute mark. - increase global mocha timeout to 5 minutes - increase pipeline time out for blob and file-share to 90 minutes. - also double the browser no activity timeout to 20 minutes
* retrieving deleted certificates without the poller * [KeyVault-Certificates] testing getDeletedCertificate * feedback by Christopher Radek * formatting * shorter name for the tests, and re-recorded things * test error messages fixed * Update sdk/keyvault/keyvault-certificates/test/CRUD.test.ts Co-Authored-By: chradek <51000525+chradek@users.noreply.github.com> * fixed the recordings Co-authored-by: chradek <51000525+chradek@users.noreply.github.com>
* Synapse data plane sdk:Initial Release * Update sdk/synapse/synapse/package.json Co-Authored-By: chradek <51000525+chradek@users.noreply.github.com> * Update sdk/synapse/synapse/src/synapseClientContext.ts Co-Authored-By: chradek <51000525+chradek@users.noreply.github.com> Co-authored-by: Dongwei Wang <bean0idear@163.com> Co-authored-by: chradek <51000525+chradek@users.noreply.github.com>
* TS samples * JavaScript samples * Changed langauge in aad sample docstring * Formatting, some doc changes for JS samples * Expanded authentication sample
* [ai-text-analytics] added note to readme about roles * Remove reference to Cognitive Services Contributor, as it also doesn't work
* Generated from 69044177e5eac1cc6897d684f262d5bb7a99617d Update code * release version 8.0.0 Co-authored-by: SDK Automation <sdkautomation@microsoft.com>
* Generated from 2d5e64e128672bf29c65595184d6f586c451abb6 fix: java tag error * release 7.0.0 Co-authored-by: SDK Automation <sdkautomation@microsoft.com>
* rename to autoRenewLockDurationInMs * generate API report * Update sdk/servicebus/service-bus/test/stress/stress_sessionState.ts * update test file * maxAutoRenewDurationInMs rename * rename to maxMessageAutoRenewLockDurationInMs * api report * update rest of seconds to ms
* Get*Options -> remove * Update sdk/servicebus/service-bus/src/serviceBusClient.ts Co-Authored-By: Ramya Rao <ramya.rao.a@outlook.com> * Update sdk/servicebus/service-bus/src/serviceBusClient.ts Co-Authored-By: Ramya Rao <ramya.rao.a@outlook.com> * remove doc strings * update param name * update retryOptions param as per Ramya's suggestions * -> createSessionReceiverOptions * fix build failures * update API report * generate API report Co-authored-by: Ramya Rao <ramya.rao.a@outlook.com>
- update instructions to obtain an access token - fix request policy factories. When factories are passed in, the credential passed to constructor is not used. - revamp browser sample to use the Parcel bundler as we no longer ship the browser distribution. * Address CR feedback
* Update changelog
* invalid param tests - update * receiveAndDelete tests - compilable * remove .only * streaming receiver tests * SBC checkpoint * revert SBC * fix recv and del test failures * update tests as per the new API * update API report * get -> create * removing un-applicable tests
* [ai-text-analytics] add validation of non-empty document array * CHANGELOG update
…ent (#8226) * Add the ability to read AZURE_AUTHORITY_HOST from environment * Add a better test skip * See if we can work around rollup issue * Move auth host function to separate file and use rollup .browser file
…iable isn't available (#8233)
azure-sdk
pushed a commit
that referenced
this pull request
Jun 9, 2020
* [MQ Goals] ARM template for Storage Blob
If the ARM template works (tests-resources.json),
then it will be the same for each of the storage clients.
tests.yml should be almost the same for the other storage clients, with
some of the content changed, reflecting the appropriate name of the
client.
* feedback
* wrong path here
* all of the tests.yml for storage
* fixed the paths
* added provisionerTenantId
* using ../archetype-sdk-tests-storage.yml again
* optimization on the env variables
* optimizations on ResourceFileLocation
* lowercase string
* variables('storageApiVersion')
* tenantId resources shouldnt be necessary
* removed provisionerTenantId and uppercased the env variables
* moved the ARM specific parameters to variables
* this is closer to work
* update
* storage-blob updated tests.yml
* seems like the archetype file needed to change
* seems like the template is invalid somehow. Trying this
* tweaks, to see if things change
* this should work now
* ResourceServiceDirectory can be optimized
* add AAD secret references in the ENV vars for the resource creation - arm-template
#RequiredForCI
* run a single test
* DEBUG #1
* reduce matrix
* only keep the samples in the pipeline
* print all
* keep test job too - Linux_Node10
* last resort - move around EnvVars
* remove duplication
* update ResourceServiceDirectory as per Daniel's suggestion
* ResourceServiceDirectory: storage
* ResourceServiceDirectory: ${{ parameters.ResourceServiceDirectory }}
* Update sdk/storage/test-resources.json
* Update sdk/storage/test-resources.json
* Update sdk/storage/test-resources.json
* remove .only
* remove env from imports
* add blob contributor role to the AAD service principal
* add "apiVersion": "[variables('authorizationApiVersion')]",
* add "authorizationApiVersion": "2017-05-01",
* 2018-01-01-preview - authorizationApiVersion
* add more sas permissions
* revert the matrix back to full
* simplified CORS rules
* smaller matrix to reduce the load
* formatting changes
* nested resources
* add cors rules for file and queue as well
* queueServices unsupported with CORS
* revert matrix back to full
* revert archetype-sdk-tests-storage.yml
* update tests.yml for storage-blob and file-share to the new format
* revert unintended change
* revert changes to archetype-storage . yml
Co-authored-by: Daniel Rodríguez <sadasant@users.noreply.github.com>
azure-sdk
pushed a commit
that referenced
this pull request
Feb 20, 2021
…afterwards, however) (Azure#13881) (NOTE: CI is still broken as we have to investigate the rest of the breakages in the test) * Problem #1 - the underlying value has changed (it used to be a CanonicalCode.OK (0) but now is StatusCode.OK (1)). * Fixing issue where the test failing wasn't actually passed on to the test framework.
azure-sdk
pushed a commit
that referenced
this pull request
Apr 24, 2022
[Hub Generated] Review request for Microsoft.Devices to add version stable/2022-02-05 (Azure#17948) * Adds base for updating Microsoft.Devices from version stable/2021-10-15 to version 2022-02-05 * Updates readme * Updates API version in new specs and examples * definition changes * example changes * no comments * swagger modelval fix #1 * general validation fix #1 * revert readonly change (breaking change) * added locatio pvtEp example * misplaced location param * location in PvtEConn response * header addition * headers #2 * example change with location sans spec * header nesting change * header under headers * header example
azure-sdk
pushed a commit
that referenced
this pull request
Aug 5, 2022
[Hub Generated] [Microsoft.Cache] 2022-05-01 API: Add "Latest" option to "redisVersion" property, async linked server, managed identity changes (Azure#19506) * Adds base for updating Microsoft.Cache from version stable/2021-06-01 to version 2022-05-01 * Updates readme * Updates API version in new specs and examples * Add 2022-05-01 API version with "redisVersion" default changes and update older API version descriptions * Sync changes * Update examples * Update example * Add Managed identity support for Import, Export and Persistence. (#1) * Update LinkedServer_Delete response code * Add x-ms-long-running-operation * Fix example and descriptions * Fix warnings Co-authored-by: MECHANDR <30642185+MECHANDR@users.noreply.github.com>
azure-sdk
pushed a commit
that referenced
this pull request
Oct 26, 2022
* Update maps dependency in pnpm-lock.yaml * Fix sample metadata errors * Fix linting errors * Fix linting errors * Fix run time error * Update package.json in samples folder * Fix linting errors * Fix linting errors * Add changelog for maps-common * Update recording files for maps-search * Add test.yml for maps-search * Add tests.yml for render and route * Update tests.yml * Update test-resources.json for maps * Fix tests.yml format error * Update env vars and remove unused export var * Remove unused code * Fix lock file * Remove route & render * Replace `beginGet*Batch` with `resume*Batch` We provide a `beginGet*Batch` method to resume the previous request previously. This is not ideal because it need to get a `batchId` which is store in the internal state of the LRO. Once the LRO brings changes, our code will be broken. We can achieve the same thing by using serialized state. This change aims to leverage this approach and create convenience methods `resume*Batch` for users * Update recording * Make the endpoint of `MapsSearchClient` configurable * Change LatLon from an interface to a tuple Also remove the `toLatLon` function since creating a tuple is quite easy. * Make the props `coordinates` and `countryCodeFilter` optional Specifying `coordinates` and `countryCodeFilter` can narrow the results of the fuzzy search. But considring the case like searching "Australia", theese fields may have no use. We decided to make them optional so user can decides if they want to narrow down the results or not. * fixup! Remove route & render * Adjust CHANGELOGs of maps-search & common Following the keyvault-common, removing CHANGELOGs from maps-common since it's not published. * Update samples * Add batchRequest sample * Add a new sample searchAddressResult * Update search.ts and its examples * enhance code example in README.md * comment unused import * Update recording for maps-search * remove redundant recording files * Add maps-search * Rename LatLong to LatLon * Fix constructor Readme doc * Update readme to new signature * Add tests partially * Change LatLon/BoundingBox to interfaces * Reduce the number of constructor overloading * Rename SearchClient as MapsSearchClient * Add options mapper tests * Add result type mapper tests (in progress) * rename clientId to mapsAccountClientId * listPolygons -> getPolygons * getPointOfInterestCategoryTree -> getPointOfInterestCategories * Expose missing model * Add alternative input file for quick experiment * Regenerate code and make changes accordingly * Reduce number of overloads of fuzzy searches * Add geojson types * Adopting new geojson definitions * Change test specs to align with new method names * clean up models and tsconfig * Expose Known* models * Remove batch sync methods * Add public tests (non-batch) * move model * Add docstrings * Start polling in begin* method * Refactor BatchPoller for retrieving batchId * Finalize batch pollers * Readd batch methods * Update generated code * Finalize batch signature * Rebuild samples & clean up * Add AAD tests * Clean up Swagger readme * Fix spelling * Add initial files for @azure/maps-route module * maps-search: change comments and perform formatting * Methods options and testing skeleton for maps-route * maps-search: refactor options * Improve convenience layer for non-LRO methods * Change variable names * Complete batch methods * update maps-search Readme * Update maps-route Readme * Add generated samples for maps-route * Update samples for maps-search * Delete backup file * Add maps-common module * Update maps-route/maps-search to use maps-common * maps-render: first draft * Use LatLon/BoundingBox from maps-common * Complete maps-render with tests * Expose required types from maps-route * Expose required types from maps-search * Complete unit tests for maps-render * Update maps-render README * Update samples * Fix comment/strings * Fix comment/strings * Update Route comment/interface * Regenerate code * Update files * Define required fields and fix code * Remove non-poller batch methods * Fix playback test failure * Rename parts of model names * Rename remaining parts of models according to review feedback * Remove TODO comments * Update Reamdme transform to reflect latest swagger * Change key policy name * Simplify batch item error model * Add brower tests * Turn maps-common into internal module * Rearrange Readme * Rewrite Readme for common * Remove batch-sync methods * Route required fields: part 1 * Route required fields: part 2 * Add AAD back * Route model fixes #1 * Route model fixes #2 + geojson named tuples * Refine geojson types * Aliased a few method options * maps-render required fields part 1 * Do not validate budget parameters * Revert "Do not validate budget parameters" This reverts commit 4ec3f5f. * Rename MapTileset to TileJson * Render: model rename & type improvement * Explicitly transform image range * Fix route record tests * Update map account creation commands in Readme * Update Readme/docstrings/samples for Search/Route/Render * Remove rollup and use dev-tool instead * Remove some empty tests * Add AAD test case and make file name shorter * Update test cases * Update cspell rules * Update recording files * Simplify test cases * Sync cspell.json from upstream * Use test-recorder 2.0 and update unit test cases * Push wrong browsers recording files * Update settings to support recording * Fix dependency errors * Upgrade api-recorder and fix errors * Code refinement * Update maps search recording files * Update karma.conf.js format * Update recording files and settings * Fix broken links * Update maps dependency in pnpm-lock.yaml * Fix sample metadata errors * Fix linting errors * Fix linting errors * Fix run time error * Update package.json in samples folder * Fix linting errors * Fix linting errors * Add changelog for maps-common * Update recording files for maps-search * Add test.yml for maps-search * Add tests.yml for render and route * Update tests.yml * Update test-resources.json for maps * Fix tests.yml format error * Update env vars and remove unused export var * Remove unused code * Fix lock file * Remove maps-render * fixup! Change LatLon from an interface to a tuple * Comment the future doc link * Update doc & fix type issue * fix lint issues * Regenerate API View * Comment the whole [apiref] since the original syntax cannot pass the Analyze/Link verification * Fix issue related to the change of maps-common - Replace beginGet*Result with resume* methods - Turn LatLon to a tuple - Add mapper to transform the string to Date because resume* method will return a serialized Date * Update test recording * Remove maps-common from the Artifacts * remove unused dependencies * Add impression in README so we can track the page views * Remove `AZURE_LOG_LEVEL` in test.yml * Update samples - Move the option parameter to the right place - Add comments & "break" for handling the type of the search result to make it clear for the user * Patch {} to properties for GeoJsonPolygon * rename maxDetourTime to maxDetourTimeInSeconds * Update recordings * Remove any * Rename to detourTimeInSeconds to align the convention * Change the test.yml EnvVar See more explanation here: - Azure#23039 (comment) - Azure#23039 (comment) * Rename numResults to numberResults * Add document comment * Add endpoint options to MapsRouteClient * Remove unused dependencies in maps-route * fixup! Change the test.yml EnvVar * Update interfaces & documents * Update changelog * Merge getRouteDirection with the additional parameter one * Update samples * Make the `maps-common to a published package * Update the import of `maps-common` in `maps-search` * Fix CI Issue * Add documents for geojson * Update the samples & readme * Leverage maps-common package in mpas-route * chore update - Remove the unnecessary rule setting of eslint - Update the legacy type path * Remove unnecessary eslint config * WIP: Remove LRO generation * WIP: refactor mapsRouteClient * WIP: first generation * Format the generated code * Support AAD * Transform the model name * Add helper functions * Add a sample - route.ts * Add samples for lro * Add test * Generate samples * Add api view * Update the generation * WIP: ReadMe * Rename "computeBestOrder" to "computeBestWaypointOrder" * Refine README * Rename model * Add test createRouteDirectionsBatchRequest * Fix lint issue * Add missing metadata * Remove maps-route/ * Update CI issue * Fix merged issue * Update version rule for maps-common * Update CI name * Remove extra lint config * Use `require` in the README for consistency * Regenerate the samples * Remove redundant recording files * Remove unused parameters in test-resources.json * Refine readme snippet * Log something specific for the result * Update samples * Split the route.ts into 3 * Remove model rename * Use default export for RLC factory function Co-authored-by: Charlie Chen <charlie.chen@microsoft.com> Co-authored-by: Charlie Chen <yuchungchen@microsoft.com> Co-authored-by: --global <alankashiwa@gmail.com>
azure-sdk
pushed a commit
that referenced
this pull request
Mar 22, 2023
[Hub Generated] Review request for Microsoft.Help to add version preview/2023-01-01-preview (Azure#23056) * Generate new Service for help * init commit microsoft.help-2023-01-01-preview * Updating swagger documentation #1 * Updating swagger documentation #2 * Adding suppressions for Help 2023-01-01-preview api * Adding suppressions for Help 2023-01-01-preview api #2 * removed unnecessary files * update examples
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Increment version for eventhubs releases