Skip to content

Fix lint checks and patch targets that look like ints#82

Merged
jg-rp merged 2 commits intomainfrom
fix-lint
Jun 22, 2025
Merged

Fix lint checks and patch targets that look like ints#82
jg-rp merged 2 commits intomainfrom
fix-lint

Conversation

@jg-rp
Copy link
Copy Markdown
Owner

@jg-rp jg-rp commented Jun 22, 2025

This PR fixes some lint checks and ensures JSON Patch targets are strings if they are operating on a mapping.

Previously,

assert JSONPatch().add(path="/1", value=99).apply({"foo": 1}) == {"foo": 1, 1: 99}

now, the new 1 is a string:

assert JSONPatch().add(path="/1", value=99).apply({"foo": 1}) == {"foo": 1, "1": 99}

@jg-rp jg-rp merged commit 6186ed0 into main Jun 22, 2025
36 checks passed
@jg-rp jg-rp deleted the fix-lint branch June 22, 2025 08:01
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