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: bug in handling gitlab-ci style urls #2527

Merged
merged 1 commit into from
Sep 28, 2021
Merged

fix: bug in handling gitlab-ci style urls #2527

merged 1 commit into from
Sep 28, 2021

Conversation

drewstinnett
Copy link
Contributor

This PR will fix parsing of the git URL format that is used by gitlab-ci runners, for example:

https://gitlab-ci-token:SOME_TOKEN@gitlab.yourcompany.com/goreleaser/goreleaser.git

The URL style above works correctly in v0.179.0, but not v0.180.0, I believe because of the following PR:

https://github.com/goreleaser/goreleaser/pull/2474/files

Thanks, for the consideration, and for the awesome project in general!

@pull-request-size pull-request-size bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Sep 27, 2021
@vercel
Copy link

vercel bot commented Sep 27, 2021

@drewstinnett is attempting to deploy a commit to the goreleaser Team on Vercel.

A member of the Team first needs to authorize it.

@caarlos0 caarlos0 added this to In progress in Board via automation Sep 28, 2021
@caarlos0 caarlos0 self-requested a review September 28, 2021 00:02
Board automation moved this from In progress to Reviewer approved Sep 28, 2021
@caarlos0 caarlos0 enabled auto-merge (squash) September 28, 2021 00:02
@caarlos0
Copy link
Member

Thanks!

@caarlos0 caarlos0 merged commit 608d48a into goreleaser:master Sep 28, 2021
Board automation moved this from Reviewer approved to Done Sep 28, 2021
@drewstinnett drewstinnett deleted the bugfix-gitlab-ci-url-detection branch September 28, 2021 21:21
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 28, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
size/S Denotes a PR that changes 10-29 lines, ignoring generated files.
Projects
No open projects
Board
Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants