Skip to content
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

Add .externalNativeBuild to Android gitignore #2054

Merged
merged 1 commit into from
Oct 3, 2016

Conversation

kevinmost
Copy link
Contributor

Reasons for making this change:

As of Android Studio 2.2 Preview 6, the default .gitignore generated for a project includes this file.

Links to documentation supporting these rule changes:

https://sites.google.com/a/android.com/tools/tech-docs/external-c-builds#TOC-Release-History

@z3ntu
Copy link

z3ntu commented Sep 28, 2016

This really should be in the Android gitignore!

@kevinmost
Copy link
Contributor Author

Absolutely should be. It's documented and the default AS .gitignore includes it now. Github's gitignore list is now out of date for Android.

@4brunu
Copy link

4brunu commented Sep 30, 2016

Is anything wrong for this PR don't get merged?
It would be really nice to have this.

@kevinmost
Copy link
Contributor Author

Nothing at all as far as I can tell, I provided ample documentation for the fact that this is now recommended to be included. Github is just ignoring this PR.

@4brunu
Copy link

4brunu commented Sep 30, 2016

Maybe they missed this PR.
@shiftkey could you please take a look a this PR?
Thanks

@shiftkey
Copy link
Member

shiftkey commented Oct 3, 2016

@4brunu @kevinmost thanks for this!

@shiftkey shiftkey merged commit c3c86c0 into github:master Oct 3, 2016
@kevinmost
Copy link
Contributor Author

👍

Thanks @shiftkey, and thanks @4brunu for tagging the right person!

@kevinmost kevinmost deleted the android-externalNativeBuild branch October 4, 2016 19:43
u9E9F pushed a commit to u9E9F/gitignore that referenced this pull request Jul 8, 2017
…uild

Add .externalNativeBuild to Android gitignore
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.

4 participants