Skip to content

Conversation

@zulquer
Copy link
Collaborator

@zulquer zulquer commented Nov 24, 2025

This pull request primarily improves code documentation consistency and enforces line length limits in the codebase. The main changes include reformatting docstrings for better readability, updating configuration files to set a maximum line length for linting tools, and making minor corrections to summary strings and function documentation.

Configuration and Linting Improvements

  • Set maximum line length to 79 characters for both Pylint and Ruff in pyproject.toml, and updated message control settings to clarify line length disabling. [1] [2]

Docstring and Documentation Formatting

Summary and Description Updates

  • Shortened and clarified module-level summary strings in several __init__.py files and other module docstrings for better accuracy and brevity. [1] [2] [3] [4] [5]

Minor Functional Correction

  • Removed an unused variable assignment in validate_amount method in src/multisafepay/api/paths/orders/request/order_request.py for cleaner code.

@codecov
Copy link

codecov bot commented Nov 24, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.53%. Comparing base (c7afd53) to head (b687423).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master      #35   +/-   ##
=======================================
  Coverage   90.53%   90.53%           
=======================================
  Files         107      107           
  Lines        2334     2334           
=======================================
  Hits         2113     2113           
  Misses        221      221           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Member

@danielcivit danielcivit left a comment

Choose a reason for hiding this comment

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

Please, check my comments in JIRA.

@zulquer zulquer closed this Nov 25, 2025
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.

3 participants