Skip to content

infra: ignore tmp folder - #3915

Merged
Shinigami92 merged 1 commit into
nextfrom
infra-ignore-tmp
Jun 24, 2026
Merged

Shinigami92 merged 1 commit into
nextfrom
infra-ignore-tmp

Conversation

@Shinigami92

Copy link
Copy Markdown
Member

Locally I develop more and more with e.g. Claude. Claude sometimes stores temporary files in a system tmp directory (e.g. /tmp on macOS or Linux). Related to some security concerns, I would like to allow Claude only to write into files inside the repo and so not write files into system directory. By ignoring the tmp folder, it can be also cleaned automatically time by time via the pnpm run clean command.

@Shinigami92 Shinigami92 added this to the vAnytime milestone Jun 24, 2026
@Shinigami92 Shinigami92 self-assigned this Jun 24, 2026
@Shinigami92
Shinigami92 requested a review from a team as a code owner June 24, 2026 17:03
@Shinigami92 Shinigami92 added the c: infra Changes to our infrastructure or project setup label Jun 24, 2026
@netlify

netlify Bot commented Jun 24, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for fakerjs ready!

Name Link
🔨 Latest commit a3f9f44
🔍 Latest deploy log https://app.netlify.com/projects/fakerjs/deploys/6a3c0de4a43e72000813eb09
😎 Deploy Preview https://deploy-preview-3915.fakerjs.dev
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@codecov

codecov Bot commented Jun 24, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.84%. Comparing base (9a1cb7d) to head (a3f9f44).

Additional details and impacted files
@@           Coverage Diff           @@
##             next    #3915   +/-   ##
=======================================
  Coverage   98.84%   98.84%           
=======================================
  Files         914      914           
  Lines        3215     3215           
  Branches      591      591           
=======================================
  Hits         3178     3178           
  Misses         33       33           
  Partials        4        4           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.
@Shinigami92
Shinigami92 enabled auto-merge June 24, 2026 17:21
@Shinigami92
Shinigami92 added this pull request to the merge queue Jun 24, 2026
Merged via the queue into next with commit 1233a4d Jun 24, 2026
24 checks passed
@Shinigami92
Shinigami92 deleted the infra-ignore-tmp branch June 24, 2026 18:53
@ST-DDT ST-DDT modified the milestones: vAnytime, v10.x Jun 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c: infra Changes to our infrastructure or project setup

3 participants