Skip to content

Do not put question mark initialization expressions behind a reference#400

Merged
Kijewski merged 2 commits intoaskama-rs:masterfrom
GuillaumeGomez:initialization-rules
Apr 12, 2025
Merged

Do not put question mark initialization expressions behind a reference#400
Kijewski merged 2 commits intoaskama-rs:masterfrom
GuillaumeGomez:initialization-rules

Conversation

@GuillaumeGomez
Copy link
Copy Markdown
Collaborator

Fixes #399.

I couldn't find any case where it would be an issue to add a reference so I suppose it's all good. :D

@GuillaumeGomez GuillaumeGomez requested a review from Kijewski April 12, 2025 19:16
Copy link
Copy Markdown
Member

@Kijewski Kijewski left a comment

Choose a reason for hiding this comment

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

@GuillaumeGomez, thank you for fixing the problem! @AMDmi3, thank you for reporting the problem!

@Kijewski Kijewski merged commit 4a17df5 into askama-rs:master Apr 12, 2025
41 checks passed
@GuillaumeGomez GuillaumeGomez deleted the initialization-rules branch April 13, 2025 08:33
@AMDmi3
Copy link
Copy Markdown

AMDmi3 commented Apr 14, 2025

Thanks a lot, this fixed my original problem!

AMDmi3 added a commit to repology/repology-rs that referenced this pull request Apr 14, 2025
Askama git contains a fix for (probably a regression in 0.13 branch)
which makes `?` operator add an extra reference level, which required
adding extra derefs in templates. With latest askama we can get rid
of these.

See askama-rs/askama#399, askama-rs/askama#400.
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.

Inconsistent (or incorrect) reference semantics of (probably) ?

3 participants