Repository navigation
test(conformance): url() without a scheme is an error, as the spec now says - #27
Merged
Merged
Conversation
…w says lightbend/config#887 put it in HOCON.md: the string inside url() is always an absolute URL, so url("foo.conf") is an error. spec/HOCON.md is the new upstream copy, RULES.md has the sentence as include-semantics-locating- resources.9 (later rules four lines down), and include-syntax/011 is no open question any more. Every parser in the report rejects it, so each gains the row: zig-hocon, typesafe/config and pyhocon measured again, hocon2, pulse-pyhocon, py.hocon and rs.hocon from their recorded output. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01CbBo1aotFTDLY2gery7qYD
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01CbBo1aotFTDLY2gery7qYD
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
lightbend/config#887 was merged on 2026-10-23. HOCON.md now says:
That settles the suite's last open question,
include-syntax/011-url-argument-must-be-a-url. zig-hocon has rejected it withInvalidUrlsince #20, so it now passes all 446 cases.Spec and rules
spec/HOCON.mdis the new upstream copy. The only difference is these four lines.RULES.mdadds the sentence asinclude-semantics-locating-resources.9, appended at the end of its section as the ids-are-stable rule asks. The 61 rules below it move four lines down, and a check confirms every rule's line number still points at its sentence.include-syntax/011: thereviewfield is gone, and the case now names bothinclude-syntax.3and the new rule.COVERAGE.mdandCOVERAGE.htmlare regenerated.Scores
rejected). Their reports still count 445 cases (no 025 row since test(conformance): a digit-initial non-number is an error #26) and stay marked as behind the suite.Docs
python/README.md: "Passes all 446 tests". The footnote says the last open question was settled by HOCON.md: the string inside url() is an absolute URL lightbend/config#887.python/README.mdstill said 444 of 445.python/MIGRATING.md: all 446.1.2.3case from test(conformance): a digit-initial non-number is an error #26, which had no line.src/Include.zig: the oracle comment on theInvalidUrltest readsspec ✓ (since lightbend/config#887). Only the comment changed, no code.Checks:
zig build testpasses,zig fmt --check srcis clean, andfill-expected.py --checkreports 446 cases with 0 sidecars stale.🤖 Generated with Claude Code
https://claude.ai/code/session_01CbBo1aotFTDLY2gery7qYD