Skip to content

chore: remove tracked platform files to improve DX (fixes #630)#631

Open
bhola-dev58 wants to merge 1 commit intoCCExtractor:mainfrom
bhola-dev58:chore/ignore-platform-files
Open

chore: remove tracked platform files to improve DX (fixes #630)#631
bhola-dev58 wants to merge 1 commit intoCCExtractor:mainfrom
bhola-dev58:chore/ignore-platform-files

Conversation

@bhola-dev58
Copy link

@bhola-dev58 bhola-dev58 commented Mar 19, 2026

Description

Hi!
I noticed that whenever we run flutter pub get or 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:

  1. Untracking those generated files from the repo history using git rm --cached.
  2. Adding their exact paths to .gitignore so they are completely ignored moving forward.

Fixes #630

Checklist

  • Code follows the established coding style guidelines
  • All tests are passing

Summary by CodeRabbit

  • Chores
    • Removed auto-generated plugin registration files for Linux, macOS, and Windows platforms from version control.
    • Updated gitignore to exclude these auto-generated files from future commits, enabling local regeneration as needed.

@coderabbitai
Copy link

coderabbitai bot commented Mar 19, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 32807e72-b333-4e49-aa9a-9140501c5b17

📥 Commits

Reviewing files that changed from the base of the PR and between f058b4a and fea7443.

📒 Files selected for processing (6)
  • .gitignore
  • linux/flutter/generated_plugin_registrant.cc
  • linux/flutter/generated_plugins.cmake
  • macos/Flutter/GeneratedPluginRegistrant.swift
  • windows/flutter/generated_plugin_registrant.cc
  • windows/flutter/generated_plugins.cmake
💤 Files with no reviewable changes (5)
  • windows/flutter/generated_plugin_registrant.cc
  • macos/Flutter/GeneratedPluginRegistrant.swift
  • windows/flutter/generated_plugins.cmake
  • linux/flutter/generated_plugins.cmake
  • linux/flutter/generated_plugin_registrant.cc

📝 Walkthrough

Walkthrough

This 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

Cohort / File(s) Summary
Build Configuration & Housekeeping
.gitignore
Added 7 gitignore entries to exclude auto-generated Flutter platform files: linux/flutter/generated_plugin_registrant.cc, linux/flutter/generated_plugins.cmake, macos/Flutter/GeneratedPluginRegistrant.swift, and Windows equivalents.
Linux Platform Build Artifacts
linux/flutter/generated_plugin_registrant.cc, linux/flutter/generated_plugins.cmake
Removed entirely auto-generated Linux plugin registration implementation and CMake build configuration that registered Flutter plugins and managed linking/bundling.
macOS Platform Build Artifacts
macos/Flutter/GeneratedPluginRegistrant.swift
Removed entirely auto-generated macOS plugin registration function that imported and registered Flutter plugins with the plugin registry.
Windows Platform Build Artifacts
windows/flutter/generated_plugin_registrant.cc, windows/flutter/generated_plugins.cmake
Removed entirely auto-generated Windows plugin registration function and CMake build configuration that managed plugin discovery, linking, and bundled library collection.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 No more generated files to track,
A cleaner repo, we're taking back!
Gitignore rules now guard the way,
While devs build freely, come what may. 🌿

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the main change: removing tracked platform files to improve developer experience and references the fixed issue.
Description check ✅ Passed The description includes a summary of changes, references the fixed issue (#630), and completes relevant checklist items (code style and tests passing).
Linked Issues check ✅ Passed The PR fully satisfies the requirements from issue #630: it adds auto-generated platform file paths to .gitignore and removes tracked files via git rm --cached.
Out of Scope Changes check ✅ Passed All changes are directly related to fixing issue #630: untracking generated platform files and updating .gitignore, with no extraneous modifications.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
📝 Coding Plan
  • Generate coding plan for human review comments

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.

Tip

Migrating from UI to YAML configuration.

Use the @coderabbitai configuration command in a PR comment to get a dump of all your UI settings in YAML format. You can then edit this YAML file and upload it to the root of your repository to configure CodeRabbit programmatically.

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.

chore: Ignore platform-generated build files to improve DX

1 participant