Skip to content

Further optimization#54

Merged
jmgasper merged 1 commit intomasterfrom
develop
Feb 25, 2026
Merged

Further optimization#54
jmgasper merged 1 commit intomasterfrom
develop

Conversation

@jmgasper
Copy link
Collaborator

No description provided.

@jmgasper jmgasper merged commit af2e743 into master Feb 25, 2026
4 checks passed
AND ($3::text[] IS NULL OR c.status::text = ANY($3::text[]))
-- exclude task challenge types from this report
AND LOWER(ct.name) NOT IN ('task', 'topgear task')
AND COALESCE(c."taskIsTask", false) = false

Choose a reason for hiding this comment

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

[❗❗ correctness]
The change from LOWER(ct.name) NOT IN ('task', 'topgear task') to COALESCE(c."taskIsTask", false) = false assumes that the taskIsTask column is correctly populated and equivalent to the previous logic. Ensure that this column accurately reflects the task status for all relevant records, as any discrepancies could lead to incorrect filtering of challenge types.

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.

1 participant