Skip to content

K hwlb poll#436

Open
TuxPowered42 wants to merge 2 commits intomainfrom
k_hwlb_poll
Open

K hwlb poll#436
TuxPowered42 wants to merge 2 commits intomainfrom
k_hwlb_poll

Conversation

@TuxPowered42
Copy link
Contributor

No description provided.

@TuxPowered42 TuxPowered42 requested a review from kofrezo March 9, 2026 11:37
Copy link
Contributor

@kofrezo kofrezo left a comment

Choose a reason for hiding this comment

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

Can you please update the title and give a bit more of a description ?

commit_query, created=created, changed=changed, deleted=deleted
commit_query,
commit_id=commit_id,
created=created, changed=changed, deleted=deleted,
Copy link
Contributor

Choose a reason for hiding this comment

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

Will this require adjustments in the signal listeners ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

At least not the ones I wrote. They access the kwargs dict directly, e.g. kwargs["changed"], they never iterate over the elements. So adding one more element should not be an issue.

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.

2 participants