search

andsens / homeshick

git dotfiles synchronizer written in bash
MIT License
2.07k stars 147 forks source link

Use HTTPS for Git clone operations in README #179

Closed Calinou closed 6 years ago

Calinou commented 6 years ago

The git:// protocol is considered insecure and deprecated, so switching to https:// is a good idea. This also capitalizes Bash and Git (which are the official casings used by those projects).

andsens commented 6 years ago

Good point, but you managed to remove the trailing spaces for the line breaks in the intro.
You should probably set your editor to not remove trailing whitespace in markdown files.

pbuckley commented 6 years ago

Any reference to where git:// is insecure/deprecated? I thought it was synonymous with ssh which is preferred over the unusable-on-the-commandline (everywhere and anywhere it counts) https.

andsens commented 6 years ago

git:// is a plaintext protocol, meaning there are no secrecy or integrity guarantees.

Calinou commented 6 years ago

@andsens Good point (trailing whitespace has a meaning in Markdown), I amended the commit.

andsens commented 6 years ago

Looks great! Merging...