Skip to content

feat(print): add print module#149

Open
laniakea2006 wants to merge 10 commits into
SAP:mainfrom
laniakea2006:main
Open

feat(print): add print module#149
laniakea2006 wants to merge 10 commits into
SAP:mainfrom
laniakea2006:main

Conversation

@laniakea2006
Copy link
Copy Markdown

Disclaimer: Do not include SAP-internal or customer-specific information in this PR (e.g. internal system URLs, customer names, tenant IDs, or confidential configurations). This is a public repository.

Description

Adds the PRINT (Print service) module to the SAP Cloud SDK for Python.

Type of Change

Please check the relevant option:

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Code refactoring
  • Dependency update

How to Test

Describe how reviewers can test your changes:

  1. Unit tests — run all tests:
    uv run pytest tests/print/ -m "not integration" -q

  2. Verify key scenarios covered by unit tests:

  • test_client.py — all CRUD, search, and retention operations; validation errors; pagination
  • test_config.py — credential loading from mounted volume and environment variables
  • test_http.py — OAuth2 token acquisition, retry, error mapping

Checklist

Before submitting your PR, please review and check the following:

  • I have read the Contributing Guidelines
  • I have verified that my changes solve the issue
  • I have added/updated automated tests to cover my changes
  • All tests pass locally
  • I have verified that my code follows the Code Guidelines
  • I have updated documentation (if applicable)
  • I have added type hints for all public APIs
  • My code does not contain sensitive information (credentials, tokens, etc.)
  • I have followed Conventional Commits for commit messages

Breaking Changes

If this PR introduces breaking changes, please describe:

  • What breaks
    None
  • Migration path for users
    No migration required.
  • Alternative approaches considered
    N/A

Additional Notes

This feature follows the existing SDK module structure described in CONTRIBUTING.md. It is intended as an additive capability and does not change existing public APIs.

@laniakea2006 laniakea2006 requested a review from a team as a code owner June 5, 2026 10:02
@cla-assistant
Copy link
Copy Markdown

cla-assistant Bot commented Jun 5, 2026

CLA assistant check
All committers have signed the CLA.

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.

1 participant