Skip to content

Remove unused Lilac and memof dependencies - #7

Merged
tiye merged 1 commit into
mainfrom
codex/remove-unused-lilac-memof
Sep 21, 2026
Merged

tiye merged 1 commit into
mainfrom
codex/remove-unused-lilac-memof

Conversation

@tiye

@tiye tiye commented Sep 21, 2026 •

Copy link
Copy Markdown
Member

Remove Lilac and memof from dependencies and the Snapshot module list; no project source definitions use either module. Also stop marking calcit.cirru as generated so future source changes stay reviewable.\n\nValidated locally with caps --strict --ci, caps verify --toolchain, Calcit fix workflow strict, Calcit check-only, dynamic-methods --max 0, JS codegen, and Vite production build. Calcit remains at 0.17.1.

Summary by CodeRabbit

  • Chores
    • Removed the Lilac and Memof modules from the project configuration.
    • Removed the corresponding Lilac and Memof dependencies.
    • Updated repository file-handling metadata for the Calcit source file.

@coderabbitai

coderabbitai Bot commented Sep 21, 2026 •

Copy link
Copy Markdown

Review Change StackReview Change Stack

Understand this PR’s impact

Explore downstream dependencies and potential security impact with Blast Radius.

View blast radius →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Advanced

Run ID: 9f83ed6b-8b10-429a-b60c-7300a4d5e08c

📥 Commits

Reviewing files that changed from the base of the PR and between 51864d1 and 618365c.

📒 Files selected for processing (3)
  • .gitattributes
  • calcit.cirru
  • deps.cirru
💤 Files with no reviewable changes (2)
  • deps.cirru
  • .gitattributes

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


Walkthrough

The project removes the lilac and memof module and dependency declarations. It also stops marking calcit.cirru as linguist-generated.

Changes

Project metadata cleanup

Layer / File(s) Summary
Remove module and dependency declarations
calcit.cirru, deps.cirru, .gitattributes
The module list and dependency file remove lilac and memof. .gitattributes no longer marks calcit.cirru as generated.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Other

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main change: removing the unused Lilac and memof dependencies. It matches the changeset and PR objective.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@tiye
tiye merged commit 3e73f30 into main Sep 21, 2026
1 check passed
@tiye
tiye deleted the codex/remove-unused-lilac-memof branch September 21, 2026 10:00
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