Skip to content

coverage: land the auditor's renewed trunc2 review - #108

Merged
kvlonge merged 3 commits into
mainfrom
fix/trunc2-review
Nov 6, 2026
Merged

kvlonge merged 3 commits into
mainfrom
fix/trunc2-review

Conversation

@kvlonge

@kvlonge kvlonge commented Nov 6, 2026

Copy link
Copy Markdown
Contributor

Lands the auditor's D118 renewal of sf.functions.trunc2 after #104. Coverage-only.

  • coverage/syntax/reviews/auditor-functions.toml: appends auditor-functions-020, copied as written from the [landing] table of coverage/syntax/audits/d118-findings-219a540d.toml (60c6cc4 on wip/syntax-omission-audit). Fingerprint 72e47302…, definition 68b34fd7… (unchanged), parser tree cc21b65e.
  • coverage/syntax/baseline.json: re-recorded with snapshot. The only change is sf.functions.trunc2 unverified → verified, so main is back to 2,388 verified forms. diff against the old baseline: 3,479 forms, 0 problems, forms gained (1): sf.functions.trunc2.

The auditor's reading: receipt ka4427f986c78 (TRUNC(1, DAY)) is errno 904 after parsing, so Snowflake resolves DAY as an identifier, and the column node is the faithful tree. That supersedes only the TRUNC part of finding d704778.

auditor-functions-020, as written in the [landing] table of
coverage/syntax/audits/d118-findings-219a540d.toml (60c6cc4 on
wip/syntax-omission-audit): the D118 re-read of PR #104 at main
219a540, renewing auditor-functions-009 for the column tree of
TRUNC's bare second argument.
Only change: sf.functions.trunc2 unverified -> verified, back to 2,388
verified forms; every other snapshot field is identical.
The auditor corrected reviewed_utc to 2026-11-06T07:26:55Z
(c11cf5b); fingerprint, definition and parser tree are unchanged.
@kvlonge
kvlonge merged commit 79ea533 into main Nov 6, 2026
@kvlonge
kvlonge deleted the fix/trunc2-review branch November 9, 2026 21:32
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