Skip to content

Commit

Permalink
[advisor] Migrate @azure/arm-advisor with @azure/dev-tool additions (#…
Browse files Browse the repository at this point in the history
…32283)

### Packages impacted by this PR

- @azure/dev-tool
- @azure/arm-advisor
- @azure/arm-agrifood
- @azure/arm-apicenter

### Issues associated with this PR

- #31338

### Describe the problem that is addressed by this PR

Migrates @azure/arm-advisor to ESM/vitest via @azure/dev-tool and adds
fixes such as `vi.fn()` support and `import "dotenv/config";`

Also adds support for ESM testing and type testing.

### 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: Jeremy Meng <jeremy.ymeng@gmail.com>
Co-authored-by: Deyaaeldeen Almahallawi <dealmaha@microsoft.com>
  • Loading branch information
3 people authored Dec 18, 2024
1 parent 1c2f325 commit 25f960a
Show file tree
Hide file tree
Showing 1,189 changed files with 76,263 additions and 77,953 deletions.
129 changes: 78 additions & 51 deletions common/config/rush/pnpm-lock.yaml

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading

0 comments on commit 25f960a

Please sign in to comment.