Skip to content

Fix format with new JuliaFormatter - #3128

Merged
blegat merged 2 commits into
masterfrom
bl/format
Nov 4, 2026
Merged

blegat merged 2 commits into
masterfrom
bl/format

Conversation

@blegat

@blegat blegat commented Nov 3, 2026 •

Copy link
Copy Markdown
Member

Looks maybe like a but in JuliaFormatter but let's just adapt

JSONSchema 1.0 permits HTTP 0.9, whose IODebug write method conflicts with the MOI file writers in the minimum-dependency ambiguity test. JSONSchema 1.1 requires HTTP 1.

Validated on Julia 1.10.12 with JSONSchema 1.1.0 and HTTP 1.0.0: General/test_errors.jl (85 tests) and FileFormats/MOF/test_MOF.jl (1761 tests). Ran JuliaFormatter 2.14.7 across the repository with no formatting changes.
Comment thread Project.toml
ForwardDiff = "1"
JSON = "0.21, 1"
JSONSchema = "1"
JSONSchema = "1.1"

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CI deliberately tests minimum dependency versions in the Julia 1.10 jobs. In ci.yml (.github/workflows/ci.yml), those jobs run julia-downgrade-compat@v2 and disable resolving to the latest compatible versions. So we need to bump JSONSchema to avoid avoiding HTTP 0.9’s ambiguous write method.

@blegat
blegat merged commit 77474f5 into master Nov 4, 2026
31 checks passed
@blegat
blegat deleted the bl/format branch November 4, 2026 16:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant