Skip to content

Fix failing CI#1383

Merged
LegNeato merged 2 commits into
graphql-rust:masterfrom
LegNeato:fix-ci-master
Jun 30, 2026
Merged

Fix failing CI#1383
LegNeato merged 2 commits into
graphql-rust:masterfrom
LegNeato:fix-ci-master

Conversation

@LegNeato

Copy link
Copy Markdown
Member

Use kebab-case git-repository-url so mdBook 0.5 accepts book.toml.
Drop the unused rustls reqwest feature whose aws-lc-sys build breaks juniper_hyper MSRV on Windows.

LegNeato added 2 commits June 29, 2026 17:39
mdBook 0.5 rejects unknown fields in the `[output.html]` table. The
field is spelled `git-repository-url` (kebab-case); the snake_case
`git_repository_url` was silently ignored by mdBook 0.4 but now fails
`mdbook build`, breaking the `deploy (Book)` CI job.
The `juniper_hyper` test harness only talks to a local plain-HTTP
server (http://127.0.0.1), so it never needs TLS. The `rustls` feature
pulled in `aws-lc-rs`/`aws-lc-sys`, whose C build fails on Windows MSVC
("C atomics require C11 or later"), breaking the
`MSRV (1.85.0, juniper_hyper, windows)` CI job. Removing the feature
drops the entire `rustls`/`aws-lc-sys` chain; only plain HTTP is used.
@LegNeato LegNeato merged commit f147793 into graphql-rust:master Jun 30, 2026
181 checks passed
@LegNeato LegNeato deleted the fix-ci-master branch June 30, 2026 01:21
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