Skip to content

Add gem badge and architecture document#23

Merged
agrberg merged 1 commit intomainfrom
ar/docs
Mar 2, 2026
Merged

Add gem badge and architecture document#23
agrberg merged 1 commit intomainfrom
ar/docs

Conversation

@agrberg
Copy link
Collaborator

@agrberg agrberg commented Mar 2, 2026

Add a RubyGems version badge to the README and create ARCHITECTURE.md, a standalone guide that explains how the codebase works: the module prepend strategy, generated method lifecycle, cache adapter system, and a file map so developers can quickly find what they need.

Add a RubyGems version badge to the README and create ARCHITECTURE.md,
a standalone guide that explains how the codebase works: the module
prepend strategy, generated method lifecycle, cache adapter system,
and a file map so developers can quickly find what they need.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds contributor-facing documentation to make the gem easier to discover and easier to work on by linking to an internal architecture guide.

Changes:

  • Added a RubyGems version badge to the README.
  • Added ARCHITECTURE.md describing Cacheable’s internal flow (prepend-based interceptor module, generated methods lifecycle, adapter resolution, memoization, and file map).
  • Linked the new architecture document from the README.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
README.md Adds gem version badge and a link to the new architecture guide.
ARCHITECTURE.md New standalone internal architecture document and file map for quicker onboarding/navigation.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@agrberg agrberg requested a review from rofreg March 2, 2026 14:16
@agrberg
Copy link
Collaborator Author

agrberg commented Mar 2, 2026

@rofreg appreciate your time and last one for a bit I swear 😅

Copy link
Contributor

@rofreg rofreg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

@agrberg agrberg merged commit c1c6a2b into main Mar 2, 2026
7 checks passed
@agrberg agrberg deleted the ar/docs branch March 2, 2026 14:29
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.

3 participants