Skip to content

Small cleanups#392

Merged
derrickstolee merged 3 commits intomicrosoft:mainfrom
derrickstolee:small-cleanups
Jun 22, 2020
Merged

Small cleanups#392
derrickstolee merged 3 commits intomicrosoft:mainfrom
derrickstolee:small-cleanups

Conversation

@derrickstolee
Copy link
Contributor

  1. @ForNeVeR reported a doc bug with the main branch which doesn't exist in microsoft/vfsforgit.
  2. @sluongng pointed out that checkout.optimizeNewBranch doesn't exist anymore. Resolves Remove checkout.optimizeNewBranch #388.
  3. @nikola-sh requested that core.autocrlf and core.safecrlf be optional. Resolves Scalar forces autocrlf=false and safecrlf=false for cloned repo #391.


The `clone` verb creates a local enlistment of a remote repository using the
[GVFS protocol](https://github.com/microsoft//VFSForGit/blob/main/Protocol.md).
[GVFS protocol](https://github.com/microsoft//VFSForGit/blob/HEAD/Protocol.md).

Choose a reason for hiding this comment

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

I'm sorry to bother you again, but could you fix a double slash here, too? // seems a bit untidy. Here, and in the index.md too.

Signed-off-by: Derrick Stolee <dstolee@microsoft.com>
Resolves microsoft#388.

Signed-off-by: Derrick Stolee <dstolee@microsoft.com>
Resolves microsoft#391.

Signed-off-by: Derrick Stolee <dstolee@microsoft.com>
@dscho
Copy link
Member

dscho commented Jun 22, 2020

The commit message of the third commit refers to a non-existing auto.crlf setting, but I think it is obvious enough that core.*crlf was meant.

@derrickstolee derrickstolee merged commit e6af600 into microsoft:main Jun 22, 2020
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.

Scalar forces autocrlf=false and safecrlf=false for cloned repo Remove checkout.optimizeNewBranch

3 participants