Skip to content

fix(pike): validate numeric bounds - #3408

Open
schani wants to merge 2 commits into
masterfrom
fix/pike-numeric-bounds
Open

fix(pike): validate numeric bounds#3408
schani wants to merge 2 commits into
masterfrom
fix/pike-numeric-bounds

Conversation

@schani

@schani schani commented Sep 1, 2026

Copy link
Copy Markdown
Member

Summary

  • enforce schema minimum and maximum constraints for numbers
  • enable the numeric-bound schema fixtures

Tests

  • npm run build
  • npm run lint
  • FIXTURE=schema-pike npm run test:fixtures -- test/inputs/schema/minmax.schema test/inputs/schema/minmax-integer.schema

@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown

Generated-output differences

7 files differ — 6 modified, 1 new, 0 deleted
263 changed lines — +263 / −0

Open the generated-output report →

@schani
schani force-pushed the fix/pike-array-map-values branch from 97a0313 to f887e5e Compare September 1, 2026 20:07
@schani
schani force-pushed the fix/pike-numeric-bounds branch from ba74ab6 to b38d578 Compare September 1, 2026 20:07
@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown

Generated-output differences

7 files differ — 6 modified, 1 new, 0 deleted
263 changed lines — +263 / −0

Open the generated-output report →

Base automatically changed from fix/pike-array-map-values to fix/pike-nested-class-decoding September 1, 2026 21:24
@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown

Generated-output differences

9 files differ — 8 modified, 1 new, 0 deleted
283 changed lines — +283 / −0

Open the generated-output report →

@schani
schani force-pushed the fix/pike-numeric-bounds branch from 765a6a0 to 51ec47a Compare September 1, 2026 21:33
@schani
schani changed the base branch from fix/pike-nested-class-decoding to master September 1, 2026 21:33
@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown

Generated-output differences

9 files differ — 8 modified, 1 new, 0 deleted
283 changed lines — +283 / −0

Open the generated-output report →

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