Skip to content

chore: add .DS_Store to scaffolded .gitignore#151

Merged
1weiho merged 1 commit into
mainfrom
claude/add-ds-store-gitignore-h7rtU
May 24, 2026
Merged

chore: add .DS_Store to scaffolded .gitignore#151
1weiho merged 1 commit into
mainfrom
claude/add-ds-store-gitignore-h7rtU

Conversation

@1weiho
Copy link
Copy Markdown
Owner

@1weiho 1weiho commented May 20, 2026

Adds .DS_Store to the .gitignore file generated by @open-slide/cli init.

Changes:

  • Updated the scaffolded .gitignore template to include .DS_Store, a macOS-specific file that should not be committed to version control

This prevents accidental commits of macOS system files in projects initialized with the CLI.

https://claude.ai/code/session_015LBf9DTVgbTjD2XptP9B1a

Summary by CodeRabbit

Bug Fixes

  • Updated project scaffolding to include .DS_Store in the generated .gitignore, preventing macOS system files from being accidentally committed to version control.

Review Change Stack

@vercel
Copy link
Copy Markdown

vercel Bot commented May 20, 2026

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

Project Deployment Actions Updated (UTC)
open-slide-demo Ready Ready Preview, Comment May 20, 2026 5:30pm
open-slide-web Ready Ready Preview, Comment May 20, 2026 5:30pm

Request Review

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 20, 2026

Walkthrough

The PR adds .DS_Store to the scaffolded .gitignore file generated by the CLI init function. A changeset entry documents this patch update to @open-slide/cli.

Changes

Gitignore .DS_Store Addition

Layer / File(s) Summary
Gitignore template and changeset update
packages/cli/src/init.ts, .changeset/scaffold-gitignore-ds-store.md
The init() function now writes .DS_Store to the scaffolded .gitignore alongside existing ignores for node_modules and dist. A changeset entry documents this patch.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

A dot-store file hops away,
Gitignore learns macOS play,
Rabbit's whiskers twitch with glee,
Cleaner repos, how fine to see! 🐰✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
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.
Title check ✅ Passed The title accurately and concisely describes the main change: adding .DS_Store to the scaffolded .gitignore file. It directly reflects the modifications in both the changeset and the init.ts file.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch

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 and usage tips.

@1weiho 1weiho changed the title Add .DS_Store to scaffolded .gitignore chore: add .DS_Store to scaffolded .gitignore May 24, 2026
@1weiho 1weiho merged commit 75667fa into main May 24, 2026
8 checks passed
@github-actions github-actions Bot mentioned this pull request May 24, 2026
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.

2 participants