Skip to content
Open
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion Adyen/services/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -12,4 +12,5 @@
from .storedValue import AdyenStoredValueApi
from .balancePlatform import AdyenBalancePlatformApi
from .disputes import AdyenDisputesApi
from .sessionAuthentication import AdyenSessionAuthenticationApi
from .sessionAuthentication import AdyenSessionAuthenticationApi
from .posMobile import AdyenPosMobileApi

Choose a reason for hiding this comment

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

medium

While this line correctly exports the AdyenPosMobileApi, there are a couple of Python conventions that could be followed to improve code quality:

  1. Alphabetical Order: For better readability and maintainability, imports should be kept in alphabetical order. posMobile should be placed before sessionAuthentication in this block of imports.
  2. File Ending: The PEP 8 style guide suggests that files should end with a single newline character. This file is currently missing it.

Please consider addressing these points for better code consistency.

References
  1. PEP 8 recommends sorting imports alphabetically within each group to improve readability and make it easier to find imports. (link)
  2. PEP 8 recommends that all files should end with a single newline character. (link)