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

build(debian): Avoid spamming .gitconfig with the same safe directory in local build #322

Merged
merged 1 commit into from
Oct 10, 2023

Conversation

EduardGomezEscandell
Copy link
Contributor

Not cool:

[safe]
        directory = C:/tools/flutter-base-dir
        directory = /mnt/c/Users/edu19/Work/ubuntu-pro-for-windows
        directory = /mnt/c/Users/edu19/Work/ubuntu-pro-for-windows
        directory = /mnt/c/Users/edu19/Work/ubuntu-pro-for-windows
        directory = /mnt/c/Users/edu19/Work/ubuntu-pro-for-windows
        ...

Copy link
Collaborator

@didrocks didrocks left a comment

Choose a reason for hiding this comment

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

More is better though, no? :)

Ok, even if there is no safe.directory entry, it will return 1 and then head to ||.

@EduardGomezEscandell
Copy link
Contributor Author

Ok, even if there is no safe.directory entry, it will return 1 and then head to ||.

Totally on purpose 😅

@EduardGomezEscandell EduardGomezEscandell merged commit 8466f74 into main Oct 10, 2023
31 of 32 checks passed
@EduardGomezEscandell EduardGomezEscandell deleted the fix-spam-gitconfig branch October 10, 2023 14:23
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