Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #38 +/- ##
==========================================
+ Coverage 90.23% 90.69% +0.45%
==========================================
Files 107 143 +36
Lines 2355 2471 +116
==========================================
+ Hits 2125 2241 +116
Misses 230 230 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull request overview
This pull request standardizes import statements across the codebase by introducing package-level __init__.py exports and refactoring example scripts to use simplified import paths. The changes improve code maintainability and readability without altering any functional behavior.
Key changes:
- Added
__all__exports to package__init__.pyfiles for cleaner public API exposure - Refactored example scripts to use package-level imports (e.g.,
from multisafepay import Sdkinstead offrom multisafepay.sdk import Sdk) - Improved code formatting in example scripts with better multi-line statement formatting
Reviewed changes
Copilot reviewed 60 out of 61 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
src/multisafepay/__init__.py |
Added Sdk export to enable top-level imports |
src/multisafepay/api/__init__.py |
Added empty __all__ list for API package |
src/multisafepay/api/base/__init__.py |
Exported AbstractManager for easier access |
src/multisafepay/api/base/listings/__init__.py |
Exported Listing and ListingPager classes |
src/multisafepay/api/base/response/__init__.py |
Exported CustomApiResponse class |
src/multisafepay/api/paths/__init__.py |
Exported all manager classes for API paths |
src/multisafepay/api/paths/auth/__init__.py |
Exported ApiToken class |
src/multisafepay/api/paths/*/request/__init__.py |
Added exports for request models across multiple endpoints |
src/multisafepay/api/paths/*/response/__init__.py |
Added exports for response models across multiple endpoints |
src/multisafepay/api/shared/__init__.py |
Exported shared API components |
src/multisafepay/api/shared/cart/__init__.py |
Exported cart-related classes |
src/multisafepay/api/shared/checkout/__init__.py |
Exported checkout-related classes |
src/multisafepay/client/__init__.py |
Exported Client and ApiKey classes |
src/multisafepay/exception/__init__.py |
Exported all exception classes |
src/multisafepay/model/__init__.py |
Exported base model classes |
src/multisafepay/util/__init__.py |
Exported Webhook utility |
src/multisafepay/value_object/__init__.py |
Exported all value object classes |
tests/multisafepay/unit/api/path/orders/request/components/test_unit_custom_info.py |
Fixed import path from customer_info to custom_info |
src/multisafepay/api/paths/orders/request/order_request.py |
Corrected import path for CustomInfo |
src/multisafepay/api/paths/orders/request/components/__init__.py |
Added CustomInfo export and fixed module name |
examples/*.py |
Updated all example scripts to use simplified imports and improved formatting |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
danielcivit
approved these changes
Dec 1, 2025
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.
This pull request refactors import statements and improves code formatting across multiple example scripts in the repository. The main goals are to standardize import styles, simplify module paths, and enhance readability by reformatting multi-line statements. No functional changes to logic or behavior are introduced.
Import statement refactoring:
from multisafepay import Sdk) and grouped related imports, reducing deep or redundant module paths in all example scripts. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12] [13]Code formatting and readability improvements:
capture_request,order_request, and API response assignments). [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12]Variable naming consistency:
gatewayListingtogateway_listing).These changes collectively make the example scripts easier to maintain and understand, especially for onboarding and future development.