Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Moving ProgressReceiver interface to models package #22345

Conversation

cochi2
Copy link
Member

@cochi2 cochi2 commented Jun 17, 2021

No description provided.

@ghost ghost added the Communication label Jun 17, 2021
@cochi2 cochi2 merged commit c6880e7 into feature/communication-servercalling-apireview Jun 17, 2021
@cochi2 cochi2 deleted the fmorales/moving_progress_reporter branch June 17, 2021 16:04
JianpingChen pushed a commit that referenced this pull request Jun 22, 2021
* first commit

* apply new name changes (#22298)

* name changes and refactoring

* name changes and refactoring

* fix(test): increase branch coverage, change groupId generation

* fix(*): remove test connection string

* fix(test): raise branch coverage

* apiview review fix (#22304)

* live test fixes (#22312)

* fix links (#22318)

* skip test refactor (#22324)

* skip test refactor (#22327)

* skip test refactor (#22328)

* api version fix

* Making ProgressReporter private. (#22331)

* apiview feedback fix (#22337)

* apiview feedback fixes

* apiview feedback fixes

* apiview feedback fixes (#22338)

* apiview feedback fixes

* Moving ProgressReceiver interface to models pacakge (#22345)

* Multiple changes. (#22346)

* Adding Functional Interface annotation

* Changing method naming

* Fixing method name (#22347)

* apiview feedback fixes

* apiview feedback fixes

* update api spec permUrl point to new swagger

* live test fix

* fix(livetest): add env variables

* fix(*): change back resource name var

* fix(*): fix issues caused by merge

* fix(*): switch live test disable var

* use latest swagger (#22394)

* use latest swagger

* use latest swagger (#22395)

* fix(live-test): disable downloadContentStreamFailure because it is failing

* fix(test): update codecoverage settings

* Multiple changes. (#22421)

* Adding download samples

* Disabling download tests for live testing

* skip code coverage

* enable code coverage

Co-authored-by: Christian Whitehead <chrwhit@microsoft.com>
Co-authored-by: Franko Morales <67804607+cochi2@users.noreply.github.com>
Co-authored-by: Zihan Zhang <43592314+zihzhan-msft@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant