Skip to content

[Core] Update Transpot __Init__ - #25344

Merged
Kashif Khan (kashifkhan) merged 9 commits into
Azure:mainfrom
kashifkhan:azcore_transport_init
Aug 3, 2022
Merged

[Core] Update Transpot __Init__#25344
Kashif Khan (kashifkhan) merged 9 commits into
Azure:mainfrom
kashifkhan:azcore_transport_init

Conversation

@kashifkhan

@kashifkhan Kashif Khan (kashifkhan) commented Jul 21, 2022

Copy link
Copy Markdown
Member

This PR is to use PEP 562 to lazy load imports and have only the base Http classes loaded by default.

  • There is a known issue with Pylint to raise an error when using getattr from pep 562
  • Some tests for storage are failing due to a ModuleNotFoundException and will look into it.

@ghost ghost added the Azure.Core label Jul 21, 2022
@kashifkhan

Copy link
Copy Markdown
Member Author

just /azp run python - core - tests

@azure-sdk

Azure SDK Bot (azure-sdk) commented Jul 21, 2022

Copy link
Copy Markdown
Collaborator

API change check

APIView has identified API level changes in this PR and created following API reviews.

azure-core
azure-core

@kashifkhan

Copy link
Copy Markdown
Member Author

/azp run python - storage - ci

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

Comment thread sdk/core/azure-core/azure/core/pipeline/transport/__init__.py Outdated
Comment thread sdk/core/azure-core/azure/core/pipeline/transport/__init__.py Outdated
Comment thread sdk/core/azure-core/azure/core/pipeline/transport/__init__.py Outdated
Comment thread sdk/core/azure-core/azure/core/pipeline/transport/__init__.py Outdated
Comment thread sdk/core/azure-core/azure/core/pipeline/transport/__init__.py Outdated
Comment thread sdk/core/azure-core/azure/core/pipeline/transport/__init__.py Outdated
Comment thread sdk/core/azure-core/azure/core/pipeline/transport/__init__.py Outdated
@kashifkhan

Copy link
Copy Markdown
Member Author

/azp run python - search- ci

@azure-pipelines

Copy link
Copy Markdown
No pipelines are associated with this pull request.

@kashifkhan

Copy link
Copy Markdown
Member Author

/check-enforcer override

@kashifkhan

Copy link
Copy Markdown
Member Author

/check-enforcer override

@kashifkhan
Kashif Khan (kashifkhan) merged commit ab99e32 into Azure:main Aug 3, 2022
@kashifkhan
Kashif Khan (kashifkhan) deleted the azcore_transport_init branch August 8, 2022 19:12
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.

5 participants