Skip to content

Remove use of Tilt::Cache #51

Closed

Description

@jeremyevans

I'm planning on deprecating Tilt::Cache in the next release of tilt (2.2.0), as it isn't used internally, isn't thread safe, and doesn't add any real value (faster to use a plain hash). grape-rabl should either copy the Tilt::Cache implementation, or switch to using a plain hash, or if you really need API compatibility for the #fetch method:

class Cache < Hash
  def fetch(*args)
    super(args) do
      self[args] = yield
    end
  end
end

Activity

  1. dblock commented on Oct 26, 2026

    @dblock
    Member

    There is now a PR addressing this: #55. It replaces Tilt::Cache with a synchronized Hash while retaining template/layout memoization, missing-layout caching, and format-specific keys. Focused tests cover operation without Tilt::Cache and concurrent cache population.

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions