Skip to content

fix(pike): validate array cardinality - #3410

Merged
schani merged 1 commit into
fix/pike-numeric-boundsfrom
fix/pike-array-cardinality
Sep 4, 2026
Merged

schani merged 1 commit into
fix/pike-numeric-boundsfrom
fix/pike-array-cardinality

Conversation

@schani

@schani schani commented Sep 1, 2026

Copy link
Copy Markdown
Member

Summary

  • enforce schema minimum and maximum item counts
  • enable the array-cardinality schema fixture

Tests

  • npm run build
  • npm run lint
  • FIXTURE=schema-pike npm run test:fixtures -- test/inputs/schema/min-max-items.schema

@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown

Generated-output differences

2 files differ — 2 modified, 0 new, 0 deleted
11 changed lines — +11 / −0

Open the generated-output report →

@schani
schani force-pushed the fix/pike-string-length branch from 17bb10e to ca93dd8 Compare September 1, 2026 20:07
@schani
schani force-pushed the fix/pike-array-cardinality branch from ba61c68 to 32ac8f1 Compare September 1, 2026 20:07
@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown

Generated-output differences

2 files differ — 2 modified, 0 new, 0 deleted
11 changed lines — +11 / −0

Open the generated-output report →

Base automatically changed from fix/pike-string-length to fix/pike-numeric-bounds September 1, 2026 21:24
@schani
schani force-pushed the fix/pike-numeric-bounds branch from 765a6a0 to 51ec47a Compare September 1, 2026 21:33
@schani
schani force-pushed the fix/pike-array-cardinality branch from 32ac8f1 to dd36317 Compare September 1, 2026 21:33
@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown

Generated-output differences

2 files differ — 2 modified, 0 new, 0 deleted
11 changed lines — +11 / −0

Open the generated-output report →

@schani
schani merged commit 6aa72d9 into fix/pike-numeric-bounds Sep 4, 2026
34 checks passed
@schani
schani deleted the fix/pike-array-cardinality branch September 4, 2026 18:08
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