Skip to content

Rename indent filter parameters #380

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

yu-iskw
Copy link
Contributor

@yu-iskw yu-iskw commented Jul 11, 2025

Resolves #379

https://jinja.palletsprojects.com/en/stable/templates/

indent(s: str, width: int | str = 4, first: bool = False, blank: bool = False) → str

@cla-bot cla-bot bot added the cla:yes label Jul 11, 2025
Signed-off-by: Yu Ishikawa <[email protected]>
@yu-iskw yu-iskw marked this pull request as ready for review July 11, 2025 12:31
@yu-iskw yu-iskw requested a review from a team as a code owner July 11, 2025 12:31
fa-assistant added a commit that referenced this pull request Jul 16, 2025
* Rename `indent` filter parameters

COPYBARA PUBLIC PR: #380

Solves #379

GitOrigin-RevId: c7c5e40

* Enrich changelog entries with PR and issue numbers

- Added PR #4389 to changelog entries
- Added linked issues if any were found

* Update to satisfy code quality check

* Code quality: remove references which are immediately dereferenced by the compiler

* Enrich changelog entries with PR and issue numbers

- Added PR #4389 to changelog entries
- Added linked issues if any were found

* Revert "Code quality: remove references which are immediately dereferenced by the compiler"

This reverts commit 43a5226c3f6bd708106cac79a75a3f712f8e99a1.

* clippy fixes

---------

Co-authored-by: Yu Ishikawa <[email protected]>
Co-authored-by: Doug Beatty <[email protected]>
Co-authored-by: Alexander Bogdanowicz <[email protected]>
Co-authored-by: Alexander Bogdanowicz <[email protected]>
GitOrigin-RevId: c04c0c0c3251a94fdd63d2678a1808fbcf25474e
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] indentparameters incompatible with standard Jinja2
1 participant