Skip to content

docs: complete aggregation function list and fix typo in aggregate.md - #6076

Merged
max-sixty merged 1 commit into
mainfrom
docs/aggregate-doc-accuracy
Jul 12, 2026
Merged

docs: complete aggregation function list and fix typo in aggregate.md#6076
max-sixty merged 1 commit into
mainfrom
docs/aggregate-doc-accuracy

Conversation

@prql-bot

Copy link
Copy Markdown
Collaborator

Problem

Two accuracy issues in aggregate.md, found during the nightly code-quality survey:

  1. Incomplete aggregation function list. The note claims "all declared aggregation functions are min, max, count, count_distinct, average, stddev and sum", but the ## Aggregate functions section of std.prql also declares all, any and concat_array. (PR docs: correct aggregation function list in aggregate.md #6002 previously corrected avgcount_distinct but didn't audit the rest of the section.)
  2. Garbled sentence. "PRQL will interpret such attempts functions as window functions" contained a stray word.

Solution

  • Add all, any and concat_array so the list matches the functions actually declared under ## Aggregate functions in std.prql.
  • Remove the stray "functions" so the sentence reads "PRQL will interpret such attempts as window functions".

Testing

Documentation-only prose change (the runnable PRQL blocks are untouched), so no test is added. Verified the list against the let declarations in std.prql and confirmed the note block keeps its manual <!-- prettier-ignore --> wrapping.

@max-sixty
max-sixty merged commit 7a37398 into main Jul 12, 2026
37 checks passed
@max-sixty
max-sixty deleted the docs/aggregate-doc-accuracy branch July 12, 2026 09:01
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