Skip to content

Update .idea/vcs.xml - #8308

Merged
youknowone merged 1 commit into
RustPython:mainfrom
bschoenmaeckers:idea-fix
Jul 18, 2026
Merged

youknowone merged 1 commit into
RustPython:mainfrom
bschoenmaeckers:idea-fix

Conversation

@bschoenmaeckers

@bschoenmaeckers bschoenmaeckers commented Jul 17, 2026 •

Copy link
Copy Markdown
Contributor

Summary

This gets added everytime I remove it. So I guess it has to stay.

Summary by CodeRabbit

  • Chores
    • Added Git version-control mapping for the project in the development environment.

@bschoenmaeckers
bschoenmaeckers marked this pull request as ready for review July 17, 2026 17:22
@coderabbitai

coderabbitai Bot commented Jul 17, 2026 •

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

The IntelliJ project configuration adds a VcsDirectoryMappings component mapping $PROJECT_DIR$ to Git.

Changes

IDE Configuration

Layer / File(s) Summary
Git directory mapping
.idea/vcs.xml
Adds a Git VCS mapping for the project root and closes the XML document.

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

Suggested reviewers: yangsijun528

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the only file changed and matches the PR's configuration update.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

@fanninpm

Copy link
Copy Markdown
Contributor

While we're at it, @youknowone could you suggest a good repository-level configuration for Zed?

@fanninpm fanninpm added the skip:ci Skip running the ci label Jul 17, 2026
@YangSiJun528

Copy link
Copy Markdown
Contributor

Thanks for catching this. I omitted it in #8262 because I wasn’t familiar with shared IDE settings and didn’t realize JetBrains IDEs would keep adding this mapping back.

@youknowone

Copy link
Copy Markdown
Member

@fanninpm I am not a Zed user, so I can't 😅

@youknowone
youknowone merged commit 732aca8 into RustPython:main Jul 18, 2026
17 checks passed
@bschoenmaeckers
bschoenmaeckers deleted the idea-fix branch July 20, 2026 11:40
youknowone pushed a commit that referenced this pull request Sep 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip:ci Skip running the ci

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants