Skip to content

Conversation

@lmazuel
Copy link
Member

@lmazuel lmazuel commented May 22, 2020

Remove an unnecessary import:

  • Not used in the file
  • The test framework will decide if identity should be imported or not depending of the live status, no direct usage should be done in any tests anyway

Caught it since it breaks my machine right now that has some identity issues. And it breaks it for no reason :)

@lmazuel lmazuel merged commit f88d011 into master May 22, 2020
@lmazuel lmazuel deleted the pointless_import branch May 22, 2020 23:33
iscai-msft added a commit that referenced this pull request May 26, 2020
…into feature/text_analytics_v3.0

* 'master' of https://github.com/Azure/azure-sdk-for-python:
  Release azure mgmt containerregistry (#11460)
  Prepare core 1.6.0 on master (#11610)
  Add force parameter to SwaggerToSdk CLI (#11609)
  LRO continuation_token (#10801)
  Remove unecessary import (#11606)
  Fix Cleanup failing on 3.8.3 (#11607)
  remove DataSourceCredentials (#11605)
  Search synonym map (#11590)
  Fix copy tests (#11594)
  Make use_training_labels positional required (#11529)
  Search refactoring 2 (#11584)
  Search refactoring 1 (#11572)
openapi-sdkautomation bot pushed a commit to AzureSDKAutomation/azure-sdk-for-python that referenced this pull request Nov 11, 2020
Fix description and samples (Azure#11606)

* update samples

* doc link

* apimsi
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.

3 participants