Skip to content

fix(exports): define __all__ in msgraph/__init__.py and fix typo in README - #1604

Open
Hardik K (hardikkaurani) wants to merge 1 commit into
microsoftgraph:mainfrom
hardikkaurani:fix/export-all-and-readme-clarification
Open

Hardik K (hardikkaurani) wants to merge 1 commit into
microsoftgraph:mainfrom
hardikkaurani:fix/export-all-and-readme-clarification

Conversation

@hardikkaurani

Copy link
Copy Markdown

Description

  • Explicitly define __all__ in msgraph/__init__.py containing __version__, GraphRequestAdapter, and GraphServiceClient.
    • Since msgraph contains a py.typed marker (PEP 561 / PEP 484), static type checkers and IDEs (such as PyLance / VSCode / PyCharm / mypy) require re-exported package symbols to be listed in __all__ to recognize them as public module exports without raising reportPrivateImportUsage warnings.
  • Fix minor grammatical typo in README.md section 2.3 ("create a GraphServiceClient object").

Copilot AI balanced review requested due to automatic review settings October 22, 2026 09:03
@hardikkaurani
Hardik K (hardikkaurani) requested a review from a team as a code owner October 22, 2026 09:03

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@hardikkaurani

Hardik K (hardikkaurani) commented Oct 22, 2026 •

Copy link
Copy Markdown
Author

Hi Adrian (@adrian05-ms),

I've opened this PR to explicitly define __all__ in msgraph/__init__.py for PEP 561 compliance, ensuring that static type checkers such as Pyright/Pylance and mypy correctly recognize re-exported symbols without raising reportPrivateImportUsage warnings.

The PR also includes a small typo fix in the README.

Whenever you have a chance, could you please take a look? Thank you for your time!

This branch has not been deployed

No deployments
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