There's some behavior change that adds parentheses for nullish coalescing in
ternary. So we will need to re-format our code base.
- remove prettier from packages that don't need them since we run the vendored
version via dev-tool
### Packages impacted by this PR
@azure/digital-twins-core
### Issues associated with this PR
#30395#29699
### Describe the problem that is addressed by this PR
Migrate test suite to use federated authentication. Skip 1 test that is
failing in the live pipeline. Tracking issue is linked to this PR
### Packages impacted by this PR
@azure/event-hubs
### Issues associated with this PR
N/A
### Describe the problem that is addressed by this PR
The service doesn't recognize the GeoDR representation of earliest
position causing tests to fail. This PR reverts this change for now
until the service is updated.
### What are the possible designs available to address the problem? If
there are more than one possible design, why was the one in this PR
chosen?
N/A
### Are there test cases added in this PR? _(If not, why?)_
N/A
### Provide a list of related PRs _(if any)_
https://github.com/Azure/azure-sdk-for-js/pull/30191/
### Command used to generate this PR:**_(Applicable only to SDK release
request PRs)_
### Checklists
- [x] Added impacted package name to the issue description
- [ ] Does this PR needs any fixes in the SDK Generator?** _(If so,
create an Issue in the
[Autorest/typescript](https://github.com/Azure/autorest.typescript)
repository and link it here)_
- [x] Added a changelog (if necessary)
### Packages impacted by this PR
@azure/eventgrid
### Issues associated with this PR
NA
### Describe the problem that is addressed by this PR
Fix Timestamp deserialization
### What are the possible designs available to address the problem? If
there are more than one possible design, why was the one in this PR
chosen?
No special design considerations
### Are there test cases added in this PR? _(If not, why?)_
No
### Provide a list of related PRs _(if any)_
https://github.com/Azure/azure-sdk-for-net/pull/44982
### Command used to generate this PR:**_(Applicable only to SDK release
request PRs)_
```autorest --typescript swagger\README.md```
### Checklists
- [X] Added impacted package name to the issue description
- [ ] Does this PR needs any fixes in the SDK Generator?** _(If so, create an Issue in the [Autorest/typescript](https://github.com/Azure/autorest.typescript) repository and link it here)_
- [X] Added a changelog (if necessary)
which only extends from tsconfig.json. There might be more customization for
packages in the early days but not any more.
This PR removes it and updates packages to extends tsconfig.json directly
***NO_CI***
### Packages impacted by this PR
@azure/identity
### Issues associated with this PR
Resolves#30119
### Describe the problem that is addressed by this PR
Updates the troubleshooting doc to call out the lack of parity in
Identity's DAC options. Most customers have the base expectation that all tier-1
languages have feature parity so we should document areas where the SDKs differ.
### Provide a list of related PRs _(if any)_
#30120
---------
Co-authored-by: Scott Addie <10702007+scottaddie@users.noreply.github.com>
### Packages impacted by this PR
@azure/identity
### Issues associated with this PR
N/A nightly failures
### Describe the problem that is addressed by this PR
on 7/1 AKS released a new patch version, but we have the older version
hardcoded. As a result, deploy-test-resources started failing.
This PR updates the test-resources-pre script to fetch the latest
available AKS version.
### Packages impacted by this PR
@azure/cosmos
### Issues associated with this PR
[Split Proof Bulk / Batch / Patch APIs in Cosmos JS SDK · Issue #18682 ·
Azure/azure-sdk-for-js
(github.com)](https://github.com/Azure/azure-sdk-for-js/issues/18682)
At present, if the bulk API SDK encounters a partition merge or split
during the process, it returns an error. Please review the process
outlined below:
### Describe the problem that is addressed by this PR
This PR ensures that the Bulk API is resistant to partition splitting
and merging.
### What are the possible designs available to address the problem? If
there are more than one possible design, why was the one in this PR
chosen?
We've implemented a BFS (breadth-first search) based approach. This
method involves splitting batch items according to the new partition
range and adding them back to the FIFO queue(newly introduced) to manage
partition splits/merges. Please refer to the following process for more
details.
### Are there test cases added in this PR? _(If not, why?)_
Yes
### Provide a list of related PRs _(if any)_
### Command used to generate this PR:**_(Applicable only to SDK release
request PRs)_
### Checklists
- [ ] Added impacted package name to the issue description
- [ ] Does this PR needs any fixes in the SDK Generator?** _(If so,
create an Issue in the
[Autorest/typescript](https://github.com/Azure/autorest.typescript)
repository and link it here)_
- [ ] Added a changelog (if necessary)
---------
Co-authored-by: Manik Khandelwal <mkhandelwal@microsoft.com>
to lint and build all packages. This can be used to valid changes that
touches
large number of files, and are skipping CIs.
---------
Co-authored-by: Ben Broderick Phillips <ben@benbp.net>
It helps old JS runtimes that only support ES 5 but we moved to ES 6
long time ago
and now moved to ES2017. Most of our packages don't have this option.
This PR
removes its usage.
### Packages impacted by this PR
@azure/monitor-opentelemetry
### Issues associated with this PR
### Describe the problem that is addressed by this PR
Live Metrics instrumentation key should still be set when an env var is
used to initialize the SDK. This functionality is also important for the
OpenTelemetry agent live metrics process.
### Checklists
- [x] Added impacted package name to the issue description
- [ ] Does this PR needs any fixes in the SDK Generator?** _(If so,
create an Issue in the
[Autorest/typescript](https://github.com/Azure/autorest.typescript)
repository and link it here)_
- [x] Added a changelog (if necessary)
### Packages impacted by this PR
@azure/-tools/vite-plugin-browser-test-map
### Issues associated with this PR
N/A
### Describe the problem that is addressed by this PR
The type declaration file wasn't set in package.json and the return type
of the plugin function was too permissive causing type errors when
imported.
### What are the possible designs available to address the problem? If
there are more than one possible design, why was the one in this PR
chosen?
N/A
### Are there test cases added in this PR? _(If not, why?)_
N/A
### Provide a list of related PRs _(if any)_
N/A
### Command used to generate this PR:**_(Applicable only to SDK release
request PRs)_
### Checklists
- [x] Added impacted package name to the issue description
- [ ] Does this PR needs any fixes in the SDK Generator?** _(If so,
create an Issue in the
[Autorest/typescript](https://github.com/Azure/autorest.typescript)
repository and link it here)_
- [ ] Added a changelog (if necessary)
### Describe the problem that is addressed by this PR
Currently, all request body that has content types that start with
"application/json" will be serialized by the `getRequestBody` function.
This is an issue for Schema Registry because the request body has
already been serialized and the content type also starts with
"application/json". Reordering the serialization method to check for
`Uint8Array` before checking for "application/json" would eliminate this
problem.
### What are the possible designs available to address the problem? If
there are more than one possible design, why was the one in this PR
chosen?
- Add an option called `skipSerialization` in the internal options bag
for sending request. It'll affect the RLC experience and force the
customer to use this option to make a successful request
- Check for specific content types. It'll be limited in scope and affect
other services that also has a similar content type format
### Packages impacted by this PR
@azure/monitor-opentelemetry
### Describe the problem that is addressed by this PR
Updates the @microsoft/applicationinsights-web-snippet package to the
latest version.
### Checklists
- [x] Added impacted package name to the issue description
- [ ] Does this PR needs any fixes in the SDK Generator?** _(If so,
create an Issue in the
[Autorest/typescript](https://github.com/Azure/autorest.typescript)
repository and link it here)_
- [x] Added a changelog (if necessary)
### Packages impacted by this PR
@azure-rest/ai-inference
### Issues associated with this PR
### Describe the problem that is addressed by this PR
Adds a test for vision model to read and describe an image url
### What are the possible designs available to address the problem? If
there are more than one possible design, why was the one in this PR
chosen?
### Are there test cases added in this PR? _(If not, why?)_
### Provide a list of related PRs _(if any)_
### Command used to generate this PR:**_(Applicable only to SDK release
request PRs)_
### Checklists
- [ ] Added impacted package name to the issue description
- [ ] Does this PR needs any fixes in the SDK Generator?** _(If so,
create an Issue in the
[Autorest/typescript](https://github.com/Azure/autorest.typescript)
repository and link it here)_
- [ ] Added a changelog (if necessary)
### Packages impacted by this PR
1. @azure/eventgrid-namespaces
2. @azure/eventgrid
### Issues associated with this PR
https://github.com/Azure/azure-sdk-for-js/issues/29699
### Describe the problem that is addressed by this PR
This PR has been created to move the eventgrid packages away from Local
Authentication. The migration guide has been used for reference.
### What are the possible designs available to address the problem? If
there are more than one possible design, why was the one in this PR
chosen?
No special design consideration.
### Are there test cases added in this PR? _(If not, why?)_
NA. Existing test cases are sufficient.
### Provide a list of related PRs _(if any)_
1. https://github.com/Azure/azure-sdk-for-js/issues/29699
2. https://github.com/Azure/azure-sdk-for-js/pull/30209
### Command used to generate this PR:**_(Applicable only to SDK release
request PRs)_
### Checklists
- [X] Added impacted package name to the issue description
- [ ] Does this PR needs any fixes in the SDK Generator?** _(If so,
create an Issue in the
[Autorest/typescript](https://github.com/Azure/autorest.typescript)
repository and link it here)_
- [ ] Added a changelog (if necessary)
- bump @azure-tools/test-credential to ^2.0.0
- remove env vars that are no longer used from tests.yml
- update karma.config.js to remove env vars
- remove unused env vars from test utils
- remove unused env vars from sample.env
- bump dev dep `@azure/identity` version to ^4.2.1
- re-publish samples
---------
Co-authored-by: Timo van Veenendaal <timov@microsoft.com>
### Packages impacted by this PR
### Issues associated with this PR
### Describe the problem that is addressed by this PR
### What are the possible designs available to address the problem? If
there are more than one possible design, why was the one in this PR
chosen?
### Are there test cases added in this PR? _(If not, why?)_
### Provide a list of related PRs _(if any)_
### Command used to generate this PR:**_(Applicable only to SDK release
request PRs)_
### Checklists
- [ ] Added impacted package name to the issue description
- [ ] Does this PR needs any fixes in the SDK Generator?** _(If so,
create an Issue in the
[Autorest/typescript](https://github.com/Azure/autorest.typescript)
repository and link it here)_
- [ ] Added a changelog (if necessary)
### Describe the problem that is addressed by this PR
This PR does two things:
- Adds a new more specific rule for api.md files inside of Mgmt packages
to ensure @qiaozha and @MaryGao can approve changes to ARM packages.
- Cleans up our manual listing of deputies for api.md reviews to be the
core review team since it was mostly redundant.
### Packages impacted by this PR
1. @azure/monitor-query
2. @azure/monitor-ingestion
### Issues associated with this PR
https://github.com/Azure/azure-sdk-for-js/issues/29699
### Describe the problem that is addressed by this PR
This PR has been created to move the monitor packages away from Local
Authentication. The migration guide has been used for reference.
### What are the possible designs available to address the problem? If
there are more than one possible design, why was the one in this PR
chosen?
None
### Are there test cases added in this PR? _(If not, why?)_
NA. Existing test cases are sufficient.
### Provide a list of related PRs _(if any)_
1. https://github.com/Azure/azure-sdk-for-js/issues/29699
2. https://github.com/Azure/azure-sdk-for-js/issues/30308
### Command used to generate this PR:**_(Applicable only to SDK release
request PRs)_
### Checklists
- [X] Added impacted package name to the issue description
- [ ] Does this PR needs any fixes in the SDK Generator?** _(If so,
create an Issue in the
[Autorest/typescript](https://github.com/Azure/autorest.typescript)
repository and link it here)_
- [ ] Added a changelog (if necessary)
---------
Co-authored-by: Minh-Anh Phan <111523473+minhanh-phan@users.noreply.github.com>
Not exactly sure what changes cause this error:
>[node-tests] Exception during run: ../storage-common/src/index.ts(4,1):
error TS2354: This syntax requires an imported helper but module 'tslib'
cannot be found.
It seems to only happen for shared source when running unit tests and
could be
related to `tsx` loader since normal tsc build works fine.
This PR change `export *` to export items explicitly.
-------
### Packages impacted by this PR
`@azure/storage-*`
We have shared source code directories like storage-common which are not
real packages. They
now cause latest `typedoc` to crash. We don't have a usage scenario so
this PR
skips directories that don't have a package name.
### Packages impacted by this PR
@azure/monitor-opentelemetry-exporter
### Issues associated with this PR
### Describe the problem that is addressed by this PR
Fix failing exporter test for logs.
### Checklists
- [x] Added impacted package name to the issue description
- [ ] Does this PR needs any fixes in the SDK Generator?** _(If so,
create an Issue in the
[Autorest/typescript](https://github.com/Azure/autorest.typescript)
repository and link it here)_
- [x] Added a changelog (if necessary)
Jeremy Meng
Minh-Anh Phan
Deyaaeldeen Almahallawi
Sarangan Rajamanickam
Azure SDK Bot
Wes Haggard
Maor Leger
Manik Khandelwal
ZiWei Chen
Jackson Weber
Matthew Podwysocki
Glenn Harper
EmmaZhu-MSFT
Scott Addie
Jeff Fisher