Skip to content
This repository was archived by the owner on Jan 24, 2025. It is now read-only.

Fix Unreal Engine plugin binaries on subfolders #417

Merged
merged 3 commits into from
Mar 1, 2022

Conversation

maximetinu
Copy link
Contributor

Pull Request

Thank you for contributing to @toptal/gitignore and https://www.gitignore.io.

New or update

Select the appropriate check box for this pull request. This helps when merging to ensure there are no conflicts with other templates or misunderstandings of how thee template list works.

New

  • Template - New .gitignore template
  • Composition - Template made from smaller templates
  • Inheritance - Template similar to an existing template
  • Patch - Template extending functionality of existing template

Update

  • [ x] Template - Update existing .gitignore template

Details

This fix is to ignore binaries and intermediate files deeper in the Plugins/ hierarchy, as can be seen in the next screenshot:

image

@maximetinu maximetinu requested a review from a team September 12, 2021 02:42
Copy link
Contributor

@deniskaber deniskaber left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @maximetinu! Thank you for contributing to gitignore.io.

In order for this change to be preserved, you have to use .patch file with your change. This way it won't be rewritten during sync with original github/gitignore.

Or you can also introduce this change to the source repository here. This way it will be automatically applied to this repo as well.

@filipechagas
Copy link
Contributor

Hello @maximetinu can you please check @deniskaber comment and adapt the PR?

@maximetinu
Copy link
Contributor Author

maximetinu commented Jan 31, 2022

Hi @filipechagas

I forgot to mention that I did what @deniskaber suggested, I opened a PR to the source repository https://github.com/github/gitignore/blob/master/UnrealEngine.gitignore so it is automatically applied to this repo as well.

This is the link to the PR: github/gitignore#3858 it's still being reviewed, it's taking long..

@filipechagas
Copy link
Contributor

Understood @maximetinu
In the meantime, you can update this PR to add these lines to a templates/UnrealEngine.patch and we'd be happy to accept the contribution.

@maximetinu
Copy link
Contributor Author

Done! ✅

I have moved the new rules to the .patch file instead 🙂

@filipechagas filipechagas dismissed deniskaber’s stale review February 24, 2022 17:52

The contributor applied the suggested change

@filipechagas
Copy link
Contributor

Hey @maximetinu I was about to merge the PR and notice there's an EOL problem. Can you please fix it?
I'm sorry I haven't noticed it before

@maximetinu
Copy link
Contributor Author

Done 🙂

@filipechagas filipechagas merged commit 9aebd5e into toptal:master Mar 1, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants