Skip to content

Conversation

@Zac-HD
Copy link
Member

@Zac-HD Zac-HD commented Dec 3, 2025

  • actions/checkout: v2 → v6
  • actions/setup-python: v1/v2 → v6
  • Python matrix: drop 3.8/3.9, add 3.13/3.14, update PyPy to 3.11
  • Regenerate all pinned deps with latest versions
  • Update ruff config to new lint section format
  • Fix mypy issues (remove stale type: ignore comments)
  • Mark known $ref/$id resolution bug as xfail

@Zac-HD Zac-HD force-pushed the claude/update-deps-actions-01GoxfiEryL6ibxZTqJpAqqE branch 2 times, most recently from d7f7f7a to b950338 Compare December 3, 2025 08:37
- actions/checkout: v2 → v6
- actions/setup-python: v1/v2 → v6
- Python matrix: drop 3.8/3.9, add 3.13/3.14, update PyPy to 3.11
- Regenerate all pinned deps with latest versions
- Update ruff config to new lint section format
- Fix mypy issues (remove stale type: ignore comments)
- Mark known $ref/$id resolution bug as xfail
- Add pre-commit config to run tox check
@Zac-HD Zac-HD force-pushed the claude/update-deps-actions-01GoxfiEryL6ibxZTqJpAqqE branch from b950338 to f83e8c4 Compare December 3, 2025 08:39
Another $ref/$id resolution bug exposed by newer jsonschema.
- Normalise -0.0 to 0.0 in get_number_bounds for consistent comparisons
- Convert whole-number float multipleOf values to int for proper type inference
- Use PyPy 3.11 in CI
The encode_canonical_json call at the start of canonicalish already
converts integer-valued floats to ints, making this branch unreachable.
PyPy treats 0 and 0.0 as different values in set operations, causing
the generator to create [0, 0.0] which jsonschema then rejects as
having non-unique elements (since 0 == 0.0 for validation purposes).
On CPython, encode_canonical_json converts integer-valued floats to ints,
but on PyPy the custom JSON encoder doesn't work (it's set to None).
Add explicit float-to-int conversion for multipleOf with pragma: no cover
since this branch is only exercised on PyPy.
@Zac-HD Zac-HD merged commit fa38b03 into python-jsonschema:master Dec 5, 2025
14 checks passed
@Zac-HD Zac-HD deleted the claude/update-deps-actions-01GoxfiEryL6ibxZTqJpAqqE branch December 5, 2025 20:41
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