Skip to content

refactor: require "cache"/"strategy" prefix on config blocks#230

Merged
alecthomas merged 1 commit intomainfrom
aat/config-prefix-blocks
Mar 27, 2026
Merged

refactor: require "cache"/"strategy" prefix on config blocks#230
alecthomas merged 1 commit intomainfrom
aat/config-prefix-blocks

Conversation

@alecthomas
Copy link
Copy Markdown
Collaborator

Cache backends must now use cache <name> { ... } and strategy
implementations must use strategy <name> { ... } instead of bare
block names. This makes the config file self-documenting and removes
the ambiguous fallthrough from cache registry to strategy registry.

Co-Authored-By: Claude Opus 4.6 (1M context) noreply@anthropic.com

Cache backends must now use `cache <name> { ... }` and strategy
implementations must use `strategy <name> { ... }` instead of bare
block names. This makes the config file self-documenting and removes
the ambiguous fallthrough from cache registry to strategy registry.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@alecthomas alecthomas requested a review from a team as a code owner March 27, 2026 04:28
@alecthomas alecthomas requested review from inez and removed request for a team March 27, 2026 04:28
@alecthomas alecthomas enabled auto-merge (squash) March 27, 2026 04:29
@alecthomas alecthomas merged commit 1eabd6d into main Mar 27, 2026
7 checks passed
@alecthomas alecthomas deleted the aat/config-prefix-blocks branch March 27, 2026 04:30
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