Skip to content

remove lowlevel client test#1303

Merged
iscai-msft merged 6 commits intoautorestv3from
remove-lowlevelclient-test
Jun 16, 2022
Merged

remove lowlevel client test#1303
iscai-msft merged 6 commits intoautorestv3from
remove-lowlevelclient-test

Conversation

@msyyc
Copy link
Member

@msyyc msyyc commented Jun 15, 2022

fix :#1274

Removing those useless test could speed our development so I think it is better to do it earlier.

@msyyc msyyc marked this pull request as ready for review June 15, 2022 02:31
@iscai-msft
Copy link
Contributor

looks good! ideally we would flatten out the version-tolerant folder in dpg, but it does make our other testing more complicated (bc we have to special case the dpg folder), so I'm fine keeping it like this for now. Thanks for your work!

@iscai-msft iscai-msft merged commit d47cccb into autorestv3 Jun 16, 2022
@iscai-msft iscai-msft deleted the remove-lowlevelclient-test branch June 16, 2022 20:09
iscai-msft added a commit that referenced this pull request Jun 23, 2022
…into full_pipeline_cadl

* 'autorestv3' of https://github.com/Azure/autorest.python: (35 commits)
  nosec bandit code (#1324)
  pin isodate to major version (#1321)
  Dont generate maxpagesize (#1320)
  reformat api-version into next link if its a client param (#1309)
  Update ChangeLog.md (#1310)
  Update ChangeLog.md
  Update package.json
  Update ChangeLog.md
  just do get call to next link for single operation dpg paging (#1307)
  Update ChangeLog.md
  remove lowlevel client test (#1303)
  default to version tolerant (#1304)
  Fix changelog date (#1302)
  hide operations with > 2 body types (#1300)
  update min dependency for `msrest` (#1301)
  remove dpg legacy generation (#1298)
  update flags (#1297)
  Update package.json
  Update ChangeLog.md
  Update ChangeLog.md
  ...
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