Skip to content

ref(issues): Drop redundant dict() re-cast in materialize_metadata#114648

Merged
mrduncan merged 1 commit intomasterfrom
mrduncan/ref/drop-redundant-dict-in-materialize-metadata
May 4, 2026
Merged

ref(issues): Drop redundant dict() re-cast in materialize_metadata#114648
mrduncan merged 1 commit intomasterfrom
mrduncan/ref/drop-redundant-dict-in-materialize-metadata

Conversation

@mrduncan
Copy link
Copy Markdown
Member

@mrduncan mrduncan commented May 1, 2026

event_type.get_metadata(...) is already wrapped in dict(...) on the line above, so the second dict(event_metadata) produces an identical copy and is immediately discarded.

`event_type.get_metadata(...)` is already wrapped in `dict(...)` on the
line above, so the second `dict(event_metadata)` produces an identical
copy and is immediately discarded. Remove it.

Co-Authored-By: Claude <noreply@anthropic.com>
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label May 1, 2026
@mrduncan mrduncan marked this pull request as ready for review May 1, 2026 23:25
@mrduncan mrduncan requested a review from a team as a code owner May 1, 2026 23:25
@mrduncan mrduncan merged commit 3fc6974 into master May 4, 2026
71 checks passed
@mrduncan mrduncan deleted the mrduncan/ref/drop-redundant-dict-in-materialize-metadata branch May 4, 2026 18:11
cleptric pushed a commit that referenced this pull request May 5, 2026
…114648)

`event_type.get_metadata(...)` is already wrapped in `dict(...)` on the
line above, so the second `dict(event_metadata)` produces an identical
copy and is immediately discarded.

Co-authored-by: Claude <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants