Skip to content

docs(idempotency): fix local_cache_max_items default in IdempotencyConfig docstring - #8731

Open
lechakrawarthy wants to merge 1 commit into
aws-powertools:developfrom
lechakrawarthy:docs/idempotency-cache-default
Open

docs(idempotency): fix local_cache_max_items default in IdempotencyConfig docstring#8731
lechakrawarthy wants to merge 1 commit into
aws-powertools:developfrom
lechakrawarthy:docs/idempotency-cache-default

Conversation

@lechakrawarthy

@lechakrawarthy lechakrawarthy commented Oct 31, 2026 •
edited
Loading

Copy link
Copy Markdown

Issue number:

closes #8729

Summary

Changes

The IdempotencyConfig docstring said local_cache_max_items defaults to 1024. The signature (local_cache_max_items: int = 256) and the docs table both use 256, so the docstring now says 256. Docstring-only change, no behaviour change.

Verification

Ran locally on Windows, Python from uv sync --locked --extra all --extra redis --extra datamasking (skipped the valkey extra because valkey-glide does not build on Windows):

uv run pytest tests/unit/idempotency tests/functional/idempotency -q
================ 202 passed, 84 warnings in 338.92s (0:05:38) =================

uv run ruff check aws_lambda_powertools/utilities/idempotency/config.py
All checks passed!

AI disclosure

This modification was implemented with the assistance of an AI-powered coding assistant. I executed the commands provided, and the resulting output is reproduced below exactly as displayed.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
@lechakrawarthy
lechakrawarthy requested a review from a team as a code owner October 31, 2026 22:52
@boring-cyborg

boring-cyborg Bot commented Oct 31, 2026

Copy link
Copy Markdown

Thanks a lot for your first contribution! Please check out our contributing guidelines and don't hesitate to ask whatever you need.
In the meantime, check out the #python channel on our Powertools for AWS Lambda Discord: Invite link
@powertools-for-aws-oss-automation powertools-for-aws-oss-automation Bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Oct 31, 2026
@lechakrawarthy
docs(idempotency): fix local_cache_max_items default in IdempotencyCo…
9140cab 
…nfig docstring

The docstring said the default was 1024; the signature and the docs table use 256.
@lechakrawarthy
lechakrawarthy force-pushed the docs/idempotency-cache-default branch from dedab2d to 9140cab Compare October 31, 2026 23:35
@sonarqubecloud

sonarqubecloud Bot commented Oct 31, 2026

Copy link
Copy Markdown

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

1 participant

@lechakrawarthy