-
Notifications
You must be signed in to change notification settings - Fork 29.7k
remove gitignored generated files #13500
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
@mravn-google can you think of any issues with removing these? |
|
Following this change, you cannot clone the Flutter repo and then launch these apps from Android Studio. You need to run It is also slightly controversial to have untracked source files like this in a source folder. In fact, I believe the intention from early Spring was to remove them from |
|
Alternatively, we should generate the plugin registrant files as part of |
|
I really like that idea. 'package get' is logically 'get dependencies' and the generated registration code is logically in that category. |
|
We should either remove these files from the repo, or add a test that verifies that the files are up to date. We keep accidentally not updating them. I don't understand why launching from an IDE can't also generate these files. It compiles the Dart code, right? Why can't it also do |
|
And if they need to be in the repo can we remove their entry from the .gitignore? |
Temporary fix while waiting for flutter#13500 to be resolved.
These files are re-generated on every flutter run, sometimes there are changes which cause annoying diffs on local machines.
These files are already gitignored in our repo [1] and I think we should just remove the files that were checked in before they were added to
.gitignore.[1] https://github.com/flutter/flutter/blob/master/.gitignore