Skip to content

fix(flags): Convert Unleash createdByUserId to str in _get_user#108578

Merged
kcons merged 2 commits intomasterfrom
kcons/targeted
Feb 20, 2026
Merged

fix(flags): Convert Unleash createdByUserId to str in _get_user#108578
kcons merged 2 commits intomasterfrom
kcons/targeted

Conversation

@kcons
Copy link
Member

@kcons kcons commented Feb 19, 2026

The createdByUserId code path returned a raw int, but FlagAuditLogRow expects created_by to be str.

@kcons kcons requested a review from a team as a code owner February 19, 2026 21:35
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Feb 19, 2026
assert items[0]["action"] == 2
assert items[0]["created_at"] == datetime(2024, 12, 30, 0, 0, tzinfo=timezone.utc)
assert items[0]["created_by"] == 1
assert items[0]["created_by"] == "1"
Copy link
Contributor

Choose a reason for hiding this comment

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

does python just need another operator, ===? :trollface:

@kcons kcons merged commit ebaa5fd into master Feb 20, 2026
78 checks passed
@kcons kcons deleted the kcons/targeted branch February 20, 2026 20:20
kcons added a commit that referenced this pull request Feb 23, 2026
#108606)

strict_equality occasionally seems a bit picky, but it flags nonsense
comparisons that can be tricky to identify otherwise.
On average, it seems to make us safer and more honest without too much
cost, and this in turn makes us able to move faster.

See also:
 * #108478
 * #108466
 * #108462
 * #108450
 * #108448
 * #108578
priscilawebdev pushed a commit that referenced this pull request Feb 24, 2026
)

The createdByUserId code path returned a raw int, but FlagAuditLogRow
expects created_by to be str.
mchen-sentry pushed a commit that referenced this pull request Feb 24, 2026
)

The createdByUserId code path returned a raw int, but FlagAuditLogRow
expects created_by to be str.
mchen-sentry pushed a commit that referenced this pull request Feb 24, 2026
#108606)

strict_equality occasionally seems a bit picky, but it flags nonsense
comparisons that can be tricky to identify otherwise.
On average, it seems to make us safer and more honest without too much
cost, and this in turn makes us able to move faster.

See also:
 * #108478
 * #108466
 * #108462
 * #108450
 * #108448
 * #108578
@github-actions github-actions bot locked and limited conversation to collaborators Mar 8, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

claude-code-assisted 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