Skip to content

Replace deprecated Tilt cache with a synchronized Hash - #55

Merged
dblock merged 1 commit into
ruby-grape:masterfrom
dblock:fix/tilt-cache-51
Oct 26, 2026
Merged

dblock merged 1 commit into
ruby-grape:masterfrom
dblock:fix/tilt-cache-51

Conversation

@dblock

@dblock dblock commented Oct 26, 2026 •

Copy link
Copy Markdown
Member

Changes

  • Replace Tilt::Cache with a plain Hash and mutex-protected cache population, keeping template/layout memoization and existing root/format-specific keys.

  • Preserve cached nil results for missing layouts and prevent duplicate template compilation under concurrent requests.

  • Add focused cases with Tilt::Cache hidden, covering template reuse, missing layouts, and concurrent population; retain existing template/layout/format cache request tests.

  • Fix Danger checks by normalizing changelog release headings, adding a README table of contents, and converting README line endings to LF.

Validation

All 34 examples and RuboCop pass locally on Ruby 4.0/Grape 4. No new dependencies or lint exclusions.

CI: Ruby matrix and lint.

Closes #51.

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

@github-actions

github-actions Bot commented Oct 26, 2026 •

Copy link
Copy Markdown

Danger Report

No issues found.

View run

@dblock dblock mentioned this pull request Oct 26, 2026
@coveralls

coveralls commented Oct 26, 2026 •

Copy link
Copy Markdown

Coverage Status

coverage: 100.0%. remained the same — dblock:fix/tilt-cache-51 into ruby-grape:master

@dblock
dblock force-pushed the fix/tilt-cache-51 branch from 3d18afc to 80fc7cf Compare October 26, 2026 13:39
Keep compiled template and missing-layout memoization without Tilt::Cache. Serialize cache population across concurrent requests and retain existing format-specific cache keys.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@dblock
dblock force-pushed the fix/tilt-cache-51 branch from 80fc7cf to 43ac827 Compare October 26, 2026 13:41
@dblock
dblock merged commit 8c8fe5f into ruby-grape:master Oct 26, 2026
13 checks passed
@dblock
dblock deleted the fix/tilt-cache-51 branch October 26, 2026 13:52
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.

Remove use of Tilt::Cache

2 participants