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

Added inclusion phrase for meta files #4355

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

Conversation

constant-flow
Copy link

@constant-flow constant-flow commented Sep 21, 2023

gitignore template of visualstudio is excluding meta file

a proposal like in this issue could make use of the inclusion phrases to make generated gitignore files more usable: toptal/gitignore.io#626

Reasons for making this change:
meta files are excluded by other gitignore templates (e.g. Visualstudio), so it's important to actively include it, so unity is working with versioning correctly.
Using this notation might help gitignore generators to improve their output. Right now the order decides which rule is more important, see the linked issue for more explanation

Links to documentation supporting these rule changes:

gitignore template of visualstudio is excluding meta file

a proposal like in this issue could make use of the inclusion phrases to make generated gitignore files more usable: toptal/gitignore.io#626
@EudenDev
Copy link

EudenDev commented Dec 10, 2023

I been using this line for a while, leaving it here so others can copy it:

# Asset meta data should only be ignored when the corresponding asset is also ignored
!/[Aa]ssets/**/*.meta

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.

2 participants