Skip to content

fix(pike): reject fractional integers - #3412

Open
schani wants to merge 1 commit into
fix/pike-string-patternfrom
fix/pike-integer-validation
Open

fix(pike): reject fractional integers#3412
schani wants to merge 1 commit into
fix/pike-string-patternfrom
fix/pike-integer-validation

Conversation

@schani

@schani schani commented Sep 1, 2026

Copy link
Copy Markdown
Member

Summary

  • reject fractional values where schemas require integers
  • cover nullable integer unions through the same validation path

Tests

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

@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown

Generated-output differences

148 files differ — 148 modified, 0 new, 0 deleted
1170 changed lines — +1170 / −0

Open the generated-output report →

@schani
schani force-pushed the fix/pike-integer-validation branch from 544f791 to 8b4bc0c Compare September 1, 2026 20:07
@schani
schani force-pushed the fix/pike-string-pattern branch from 704dd44 to 24fb398 Compare September 1, 2026 20:07
@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown

Generated-output differences

148 files differ — 148 modified, 0 new, 0 deleted
1170 changed lines — +1170 / −0

Open the generated-output report →

@schani
schani force-pushed the fix/pike-string-pattern branch from 24fb398 to 05ec989 Compare September 1, 2026 21:29
@schani
schani force-pushed the fix/pike-integer-validation branch from 8b4bc0c to 08ce35e Compare September 1, 2026 21:29
@schani
schani force-pushed the fix/pike-string-pattern branch from 05ec989 to 4c55d98 Compare September 1, 2026 21:33
@schani
schani force-pushed the fix/pike-integer-validation branch from 08ce35e to 3990323 Compare September 1, 2026 21:34
@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown

Generated-output differences

148 files differ — 148 modified, 0 new, 0 deleted
1170 changed lines — +1170 / −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