Skip to content

Conversation

behnazh-w
Copy link
Member

Summary

This PR addresses a bug related to the uncaught SourceCodeError exception in the PyPI type stub file analyzer. An additional integration test has been added to ensure correct handling for Python packages distributed only as wheels.

Description of changes

  • Bug fix: handles situations where source code files cannot be downloaded. In this case, the TypeStubFileAnalyzer fails, instead of raising an exception.
  • A TODO comment is added to note that .pyi stub files may reside in both sdist and wheel distributions.
  • Unit test updated.
  • Added a new integration test case and a policy for packages only distributed as wheels and not as sdists (e.g., ibm-agent-analytics-common).

Signed-off-by: behnazh-w <behnaz.hassanshahi@oracle.com>
@behnazh-w behnazh-w requested a review from tromai as a code owner September 23, 2025 01:06
@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Sep 23, 2025
@behnazh-w behnazh-w requested review from art1f1c3R and removed request for tromai September 23, 2025 01:06
Signed-off-by: behnazh-w <behnaz.hassanshahi@oracle.com>
@behnazh-w behnazh-w force-pushed the behnazh/fix-mal-chk-exception branch from 63b671a to fa655eb Compare September 25, 2025 01:25
@behnazh-w behnazh-w merged commit e6e3266 into main Sep 25, 2025
10 of 11 checks passed
Demolus13 pushed a commit to Demolus13/macaron that referenced this pull request Sep 26, 2025
This PR addresses a bug related to the uncaught SourceCodeError exception in the PyPI type stub file analyzer. An additional integration test has been added to ensure correct handling for Python packages distributed only as wheels.

Signed-off-by: behnazh-w <behnaz.hassanshahi@oracle.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OCA Verified All contributors have signed the Oracle Contributor Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants