Moving to a modular bicep definition so that we can share bits and
pieces between all of them. This PR factors out the key vault definition
so other bicep scripts can import it too. For now, it lives next to the
event hub's test-resources.bicep script but I'll consider moving it to a
central place once other libraries move to import it.
The PR also edits the script to make it more compact.
Live tests run:
https://dev.azure.com/azure-sdk/internal/_build/results?buildId=4079360&view=results
- adjust options passed `RuleTester` constructor
- remove `recommended` property that no longer exists
- use `output` of array to indicate multi-pass fixes with overlapping
range
- temporarily disable two test cases of ts-modules-only-named suite.
***NO_CI***
- bump typescript-eslint dependencies version to ~8.2.0
- [eslint-plugin] suppress new linting errors appearing with v8
- [eslint-plugin] react to removed typescript-eslint property
related PR https://github.com/typescript-eslint/typescript-eslint/pull/9025
- [eslint-plugin] turn off @typescript-eslint/no-unused-expressions
as no-unused-expressions is already reported by eslint
- [dev-tool] fix new linting errors
- Remove @typescript-eslint/ban-types that no longer exists
and replace with more specific rules wherever applicable.
- [cosmosdb] combine the test eslint config with the main one
- delete catch expression when caught error is not used
- fix or suppress linting errors in code files
### 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)
---------
Co-authored-by: sorrego <sorrego@microsoft.com>
Co-authored-by: sorrego-msft <113116666+sorrego-msft@users.noreply.github.com>
### Packages impacted by this PR
@azure-rest/ai-inference
### Issues associated with this PR
### Describe the problem that is addressed by this PR
add npm run generate command to gen from typespec
### 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
@azure/microsoft-playwright-testing
### Issues associated with this PR
Uncomment links to other README files in the
microsoft-playwright-testing SDK.
### Describe the problem that is addressed by this PR
As per
https://github.com/Azure/azure-sdk/blob/main/docs/policies/README-TEMPLATE.md#link-guidelines,
any relative link to a file which does not exist in main, will require
the initial PR to go in with link commented out, followed by a PR to
uncomment those links.
### 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?)_
Not pplicable
### Provide a list of related PRs _(if any)_
Initial PR - https://github.com/Azure/azure-sdk-for-js/pull/30433
### 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
@azure/keyvault-certificates
### Issues associated with this PR
N/A
### Describe the problem that is addressed by this PR
Migrates keyvault-certificates to ESM and tshy
### Packages impacted by this PR
@azure/identity
### Issues associated with this PR
Resolves https://github.com/Azure/azure-sdk-for-js-pr/issues/262
### Describe the problem that is addressed by this PR
Adds support for providing an objectID to be used in managed Identity.
### Packages impacted by this PR
@azure/identity
### Issues associated with this PR
N/A - test fixes
### Describe the problem that is addressed by this PR
Identity client libraries need some extensive setup to test various
credentials in different workloads. Some of the commands
require the az cli, and this enables us to login using az cli.
Eventually, my hope is to use this to provide federated credentials for
keyvault managed HSMs resource creation where our credential expires by
the time the HSM is deployed
Deyaaeldeen Almahallawi
Azure SDK Bot
Jeremy Meng
Jeremy Meng
Daniel Ortega
Glenn Harper
ZiWei Chen
Siddharth Singha Roy
Maor Leger