chore: remove tracked platform files to improve DX (fixes #630)#631
chore: remove tracked platform files to improve DX (fixes #630)#631bhola-dev58 wants to merge 1 commit intoCCExtractor:mainfrom
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (6)
💤 Files with no reviewable changes (5)
📝 WalkthroughWalkthroughThis PR adds gitignore rules for auto-generated Flutter platform-specific build files and removes those files from version control. The changes ensure that locally-generated plugin registrant files and CMake build configurations for Linux, macOS, and Windows are not tracked by Git, reducing merge conflicts and dirty working directories. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
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. Comment Tip Migrating from UI to YAML configuration.Use the |
Description
Hi!
I noticed that whenever we run
flutter pub getor test the app locally, Flutter auto-generates some platform-specific files for Linux, macOS, and Windows. Because some of these were unfortunately tracked in the repo, they constantly dirty everyone's working directory, forcing contributors to manually discard them before every commit just to keep PRs clean.This PR fixes that annoying DX issue by:
git rm --cached..gitignoreso they are completely ignored moving forward.Fixes #630
Checklist
Summary by CodeRabbit