Skip to content

Allow defined tables without exact rows - #415

Merged
Alex Vallone (amv146) merged 4 commits into
masterfrom
alexvallone/nullable-defined-table-rows
Aug 25, 2026
Merged

Allow defined tables without exact rows#415
Alex Vallone (amv146) merged 4 commits into
masterfrom
alexvallone/nullable-defined-table-rows

Conversation

@amv146

@amv146 Alex Vallone (amv146) commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Summary

  • allow DefinedTable to retain a known column schema when exact rows are unavailable
  • make the rows property optional while preserving rows: [] as an exact empty table
  • allow callers to omit the definedTable factory's third argument
  • continue validating the shape and types of every supplied exact row
  • distinguish unknown rows from exact empty rows during type equality checks
  • bump @microsoft/powerquery-parser to version 2.0.0 for the breaking type contract change

Testing

  • npm run build
  • targeted type utility tests
  • targeted ESLint
  • full library test suite (690 passing, 1 pending)
  • full resource test suite (102 passing)
  • npm pack --dry-run --json

Alex Vallone and others added 4 commits August 25, 2026 13:05
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 946bd35e-0c84-49f0-90ff-a15d522d34f8
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 946bd35e-0c84-49f0-90ff-a15d522d34f8
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 946bd35e-0c84-49f0-90ff-a15d522d34f8
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 946bd35e-0c84-49f0-90ff-a15d522d34f8
@amv146
Alex Vallone (amv146) merged commit 1e49dd1 into master Aug 25, 2026
5 checks passed
@amv146
Alex Vallone (amv146) deleted the alexvallone/nullable-defined-table-rows branch August 25, 2026 20:56
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