Skip to content

Make the tests pass#2

Merged
arifibrahim4 merged 4 commits intoarifibrahim4:feature/sms-core-v2-and-test-recoder-migrationfrom
petrsvihlik:petrsvihlik/sms-migration-v2
Apr 25, 2022
Merged

Make the tests pass#2
arifibrahim4 merged 4 commits intoarifibrahim4:feature/sms-core-v2-and-test-recoder-migrationfrom
petrsvihlik:petrsvihlik/sms-migration-v2

Conversation

@petrsvihlik
Copy link
Copy Markdown

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 repository and link it here)
  • Added a changelog (if necessary)

@petrsvihlik petrsvihlik changed the title minor fixes Make the tests pass Apr 22, 2022
@arifibrahim4 arifibrahim4 merged commit 1efbe0f into arifibrahim4:feature/sms-core-v2-and-test-recoder-migration Apr 25, 2022
arifibrahim4 added a commit that referenced this pull request Apr 26, 2022
* sms core v2 migration changes

* fixed build issue

* unit test fixes

* record test mode fix

* recording updated

* migrated to new test recorder version

* Make the tests pass (#2)

format and test fixes

* recording update & code owner update

* analyzer fix

* update code owner for sms

Co-authored-by: Petr Švihlík <rocky.intel@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants