Skip to content

fix(glrepo): Remove side effect from RemoteURL

Kerri Miller requested to merge github/fork/zemzale/fix-group-repo-clone into trunk

Created by: zemzale

Description

RemoteURL was changing one if it's passed in variables. This introduced problems with subsequent calls to the function, because member RemoteArgs.Protocol was changed to from https to https://. This made every other call to the function skip over the protocol and just return ssh URL.

Related Issue

Resolves #710 (closed)

How Has This Been Tested? Localy tested and unit tests passed

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation
  • Chore (Related to CI or Packaging to platforms)

Merge request reports