Conversation
- Fix --format text example and status table to match actual output (multi-line per-feature blocks, real support level values incl. unknown) - Add guide: contributing a new server profile to caldav/compatibility_hints.py - Add guide: storing checker results in ~/.config/caldav/calendar.conf (named profile, inline features, and base+overrides patterns) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
The caldav test server registry now registers servers with capitalised names (e.g. "Radicale", "Xandikos") after a recent refactor in the caldav library. The exact-match lookup therefore failed silently when users passed lowercase names like --name radicale. Add a case-insensitive fallback that iterates registry.all_servers() when the exact key is not found. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…ing) Two bugs prevented --name from finding servers in the caldav test registry: 1. The caldav library capitalised its server names (Radicale, Xandikos). The registry lookup now falls back to a case-insensitive search when the exact key is not found. 2. The caldav-server-tester's own tests/ package shadows the caldav project's tests/test_servers module — either via sys.modules (if already imported) or via '' (CWD) in sys.path. Switch from `from tests.test_servers import …` to loading the module directly with importlib.util.spec_from_file_location, bypassing sys.path resolution entirely. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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
1 participant
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.
Summary
--format textsection: update the example output and status table to match the actual current multi-line per-feature format, replace old[ok]/[no]/[error]markers with real support level values (full,unsupported,quirk,fragile,broken,unknown), and explain the distinction between "Extra check information" (observed behaviour) and "Description of the feature" (general definition)caldav/compatibility_hints.pyvia a PR to the caldav project~/.config/caldav/calendar.conf, covering the named-profile, inline-features, and base+overrides patternsTest plan
🤖 Generated with Claude Code