Skip to content

Support request-aware template roots - #58

Merged
dblock merged 1 commit into
ruby-grape:masterfrom
dblock:fix/dynamic-template-roots-25
Oct 27, 2026
Merged

dblock merged 1 commit into
ruby-grape:masterfrom
dblock:fix/dynamic-template-roots-25

Conversation

@dblock

@dblock dblock commented Oct 27, 2026 •

Copy link
Copy Markdown
Member

Changes

Support static configuration roots and zero-argument Proc/lambda roots evaluated in the current Grape endpoint context. Existing request roots take precedence, version directories remain opt-in, and fallback responses do not resolve templates. Cover per-request template/layout cache isolation and explicit resolver failures.

Validation

Local checks cover 41 examples, including version-specific templates and layouts with caching enabled and disabled, explicit request-root precedence, resolver errors, and formatter fallback. Hosted CI passes the full Ruby/Grape matrix and lint; Danger also passes.

Closes #25.

Created by Copilot, an AI, on behalf of @dblock.

@dblock
dblock force-pushed the fix/dynamic-template-roots-25 branch from 847f581 to c31531d Compare October 27, 2026 02:37
@github-actions

github-actions Bot commented Oct 27, 2026 •

Copy link
Copy Markdown

Danger Report

No issues found.

View run

@coveralls

coveralls commented Oct 27, 2026 •

Copy link
Copy Markdown

Coverage Status

coverage: 100.0%. remained the same — dblock:fix/dynamic-template-roots-25 into ruby-grape:master

Resolve configured roots in the current endpoint context, preserve explicit request roots, and isolate cached templates and layouts by resolved path. Keep static roots and unversioned paths unchanged.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@dblock
dblock force-pushed the fix/dynamic-template-roots-25 branch from c31531d to 7ed6917 Compare October 27, 2026 02:40
@dblock
dblock merged commit 763a360 into ruby-grape:master Oct 27, 2026
22 checks passed
@dblock
dblock deleted the fix/dynamic-template-roots-25 branch October 27, 2026 09:57
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.

Dynamically configure template path

2 participants