[Core] Fix pylint unused-import warnings for eager msal/requests imports in identity.py - #34001
Draft
Aditya Pujara (a0x1ab) with Copilot wants to merge 3 commits into
Draft
[Core] Fix pylint unused-import warnings for eager msal/requests imports in identity.py#34001Aditya Pujara (a0x1ab) with Copilot wants to merge 3 commits into
Aditya Pujara (a0x1ab) with Copilot wants to merge 3 commits into
Conversation
…during CLI startup to avoid Python 3.14 module-lock deadlock * Initial plan * [Core] Eagerly import requests/msal in auth/identity.py to avoid Python 3.14 deadlock Co-authored-by: a0x1ab <59631311+a0x1ab@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: a0x1ab <59631311+a0x1ab@users.noreply.github.com>
Co-authored-by: a0x1ab <59631311+a0x1ab@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Fix pylint warnings for unused import in identity.py
[Core] Fix pylint unused-import warnings for eager msal/requests imports in identity.py
Aug 27, 2026
Collaborator
|
Core |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🤖 PR Validation — ️✔️ All clear
Related command
N/A (core startup path, not a user-facing command)
Description
The eager
import msal/import requestsadded toidentity.py(to pre-load modules on the main thread and avoid Python 3.14 import-lock deadlocks) triggered pylintW0611: Unused importfailures inazdev style, since the names are not directly referenced in the module.# pylint: disable=unused-importinline on both eager-import lines, alongside the existing# noqa: F401flake8 suppression:The eager-import behavior and intent are fully preserved; only the linter suppression is added.
Testing Guide
Confirm
Your code has been rated at 10.00/10with noW0611errors.History Notes
{Core} Fix pylint unused-import warnings for eager msal/requests imports
This checklist is used to make sure that common guidelines for a pull request are followed.
The PR title and description has followed the guideline in Submitting Pull Requests.
I adhere to the Command Guidelines.
I adhere to the Error Handling Guidelines.