Skip to content
This repository has been archived by the owner on Nov 22, 2022. It is now read-only.

respect GITLAB_HOST when resolving remotes #500

Merged
merged 1 commit into from
Jan 3, 2021
Merged

Conversation

profclems
Copy link
Owner

@profclems profclems commented Jan 3, 2021

When GITLAB_HOST or GITLAB_URI has been set, we resolve only local remotes that match the set hostname.
With this in place, a heterogeneous mix of hosts in a local remote file will not cause GITLAB_HOST to be ignored.

An error is returned if the hostname specified in GITLAB_HOST is not present in the local remotes.

Resolves #499

When GITLAB_HOST or GITLAB_URI has been set, we resolve only local remotes that match the set hostname.
With this in place, a heterogeneous mix of hosts in a local remote file will not cause GITLAB_HOST to be ignored.

An error is returned if the hostname specified in GITLAB_HOST is not present in the local remotes.
@codecov
Copy link

codecov bot commented Jan 3, 2021

Codecov Report

Merging #500 (b438495) into trunk (3662eb3) will decrease coverage by 0.02%.
The diff coverage is 36.36%.

Impacted file tree graph

@@            Coverage Diff             @@
##            trunk     #500      +/-   ##
==========================================
- Coverage   55.90%   55.87%   -0.03%     
==========================================
  Files          83       83              
  Lines        5552     5562      +10     
==========================================
+ Hits         3104     3108       +4     
- Misses       2120     2125       +5     
- Partials      328      329       +1     
Impacted Files Coverage Δ
commands/cmdutils/factory.go 0.00% <0.00%> (ø)
commands/cmdutils/remote_resolver.go 54.54% <57.14%> (+0.49%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3662eb3...b438495. Read the comment docs.

@profclems profclems merged commit 770a395 into trunk Jan 3, 2021
@profclems profclems deleted the fix-gitlab-host-env branch January 3, 2021 22:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow GITLAB_HOST to select between hosts with mixed git remotes
1 participant