CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

I have read the CLA Document and I hereby sign the CLA

Thanks for the contribution! I do wonder whether or not we should do this, though. This will introduce a local change into the repo being checked out (git tracks executable bit on files). Another solution would be to chmod +x bootstrap.sh && git add bootstrap.sh && git commit in your personal repo.

Perhaps it would be safer to check the executable bit and print a pretty error explaining how to fix it? We could also run scripts via interpreter like /bin/sh, etc, but this would also require checking potential shebang (#!/bin/other) at the top of the script. And doing that might also not work properly on Windows.

Hmm.. Let me look into it

This will introduce a local change into the repo being checked out (git tracks executable bit on files). Another solution would be to chmod +x bootstrap.sh && git add bootstrap.sh && git commit in your personal repo.

Yes, you are correct in that case.

So, I found this on StackOverflow.

Shall I set a git config, so that such changes are ignored, for the dotfiles cloned repo?

Perhaps it would be safer to check the executable bit and print a pretty error explaining how to fix it?

Yes, can be done too.

I am unable to choose which would be a better solution.

Yeah on second thoughts, 2nd solution would be better. Check for exec bits, and notify user about the same. I am changing this, and pushing

@mafredri Can you review it!

I will also try to update docs.

@mafredri I have added to docs about the same

Added all suggested changes.