Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: #479 allow nested repos for gitlab.com #494

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

tvizor
Copy link

@tvizor tvizor commented Jun 3, 2024

Address #479
As per GitLab documentation, repositories can be nested under projects and subgroups: https://docs.gitlab.com/ee/user/group/subgroups/

MR is to remove the limit of 3 components of the path (gitlab.com/username/repo) and allow referencing paths to directories inside the repo.

Parse must be performed on the whole URL before evaluating and adding the .git suffix to the repo name to keep the ref query in its original format (?ref=master vs %3Fref=master if parsed as a part of the path).

Copy link

hashicorp-cla-app bot commented Jun 3, 2024

CLA assistant check
All committers have signed the CLA.

@tvizor tvizor force-pushed the allow-nested-repos-gitlab-com-allow-subpath-query branch 2 times, most recently from b1ac42c to a76aa0f Compare June 3, 2024 19:00
@tvizor tvizor force-pushed the allow-nested-repos-gitlab-com-allow-subpath-query branch from a76aa0f to 5515e0a Compare June 3, 2024 20:09
@tvizor tvizor changed the title fix: allow nested repos for gitlab.com fix: #479 allow nested repos for gitlab.com Jun 3, 2024
@gmaghera
Copy link

gmaghera commented Jun 7, 2024

Could we get someone to review this change, please?

@tvizor tvizor force-pushed the allow-nested-repos-gitlab-com-allow-subpath-query branch from b92f938 to 83ccf98 Compare July 16, 2024 22:06
@crw
Copy link
Contributor

crw commented Jul 17, 2024

I am not sure why I am only now seeing this notification in GitHub, but I will raise this in triage. I find this comment interesting, gruntwork-io/terragrunt#3031 (comment), will see what is underneath that difference. To set expectations, there is a strong headwind to getting PRs reviewed and merged into this repo due to multiple products using it and an unwillingness to inadvertently add new bugs. In any case, thanks for the submission!

@crw crw linked an issue Aug 5, 2024 that may be closed by this pull request
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.

Unable to download group gitlab repository files
3 participants