Skip to content

chore: ignore .trunk/tools cache directory#149

Open
dannyneira wants to merge 2 commits into
mainfrom
danny/gitignore-trunk-tools
Open

chore: ignore .trunk/tools cache directory#149
dannyneira wants to merge 2 commits into
mainfrom
danny/gitignore-trunk-tools

Conversation

@dannyneira
Copy link
Copy Markdown
Member

Summary

Adds .trunk/tools to .gitignore to prevent the Trunk tools cache directory from being tracked.

Changes

.gitignore

  • Added .trunk/tools under a new "Trunk tools cache" comment

Co-Authored-By: Oz oz-agent@warp.dev

Co-Authored-By: Oz <oz-agent@warp.dev>
@cla-bot cla-bot Bot added the cla-signed label May 27, 2026
@vercel
Copy link
Copy Markdown

vercel Bot commented May 27, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview, Comment May 27, 2026 11:40pm

Request Review

@oz-for-oss
Copy link
Copy Markdown
Contributor

oz-for-oss Bot commented May 27, 2026

@dannyneira

I'm starting a first review of this pull request.

You can view the conversation on Warp.

I completed the review and no human review was requested for this pull request.

Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz

Copy link
Copy Markdown
Contributor

@oz-for-oss oz-for-oss Bot left a comment

Choose a reason for hiding this comment

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

Overview

This PR updates .gitignore to ignore the local .trunk/tools cache directory.

Concerns

  • None found. The change is limited to ignoring a local tool cache path and does not introduce code or security-relevant behavior.

Verdict

Found: 0 critical, 0 important, 0 suggestions

Approve

Comment /oz-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz

Copy link
Copy Markdown
Contributor

@rachaelrenk rachaelrenk left a comment

Choose a reason for hiding this comment

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

LGTM! 🛸

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants