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

UE appropriate .gitignore & '/Intermediate' removal #1

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Tolc
Copy link

@Tolc Tolc commented Jun 25, 2022

Hi!
I've been following your YouTube videos for quite some time and I must say your TrueFPS project is very interesting so when I saw you added this plugin on github I cloned it immediately.

Doing so, I've noticed that you pushed your /Intermediate folder, and that the .gitignore file was empty (which led to this unnecessary folder being committed).
In short, this is an issue because those UE-generated folders can be really heavy (so github may rightfullly just not accept your push because of its files sizes), and they are not intended to be copied from a computer to another (this is really prone to causing build errors and such).

So I took the liberty of fixing that on my side, and made this pull-request so that it is easy for you to merge it.
Basically I've just added an Unreal Engine appropriate configuration in the .gitignore file so that git does not add any UE-generated folder in the future, and just straight up deleted /Intermediate from versioning.

Have a nice day, and thank you for your work for the UE community!

PS: I'm the same Tolc that wrote to you under this video (https://www.youtube.com/watch?v=K9eiGsT3cqE)

Removed unnecessary /Intermediate folder accordingly
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.

1 participant