Skip to content

Fix README skills path and repo URLs; add .gitignore#8

Closed
CloudAI-X wants to merge 1 commit into
mainfrom
devin/1783629647-fix-readme-paths-gitignore
Closed

Fix README skills path and repo URLs; add .gitignore#8
CloudAI-X wants to merge 1 commit into
mainfrom
devin/1783629647-fix-readme-paths-gitignore

Conversation

@CloudAI-X

Copy link
Copy Markdown
Owner

Summary

Fixes documentation bugs introduced in the README and prevents .DS_Store from recurring — findings from a code review of the 5 most recent commits authored by @CloudAI-X (they had no authored PRs to review).

README.md

  • Skills live in skills/, not .claude/skills/. Corrected the directory in the Installation, How It Works, and Contributing sections so the instructions actually match the repo layout.
  • Installation clone/submodule commands pointed at github.com/pinkforest/threejs-playground.git; changed to github.com/CloudAI-X/threejs-skills.git.

.gitignore (new)

  • Commits e2b7cff and 260ead4 deleted .DS_Store files but no ignore rule existed, so they would be recommitted. Adds .DS_Store / **/.DS_Store.

Not included (flagged in review, left as-is): the README still refers to "Claude Code" throughout while acknowledgments were trimmed, and the "audited against r160+" claim is drifting (PR #6 bumps to r183) — happy to follow up if wanted.

Link to Devin session: https://app.devin.ai/sessions/ff96459e5b524382a21d7979f7b68790

Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@CloudAI-X CloudAI-X self-assigned this Jul 9, 2026
@devin-ai-integration

Copy link
Copy Markdown

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment, CI, and merge conflict monitoring

@CloudAI-X CloudAI-X closed this Jul 9, 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.

1 participant