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(Gitlab): slug_regex when url has different host #422

Conversation

ArmandoAssuncao
Copy link
Contributor

@ArmandoAssuncao ArmandoAssuncao commented Mar 11, 2022

Sometimes the host of runner in Gitlab can be different of api_endpoint.

A example:

  • api_endpoint is https://gitlab.com/api/v4
  • runner host is https://git-us-east1-d.ci-gateway.int.gprd.gitlab.net:8989/group/x/y/z.git

the difference is in gitlab.com and gitlab.net

This PR treat when hosts are differents.

@ArmandoAssuncao ArmandoAssuncao requested a review from a team as a code owner March 11, 2022 02:37
@ArmandoAssuncao ArmandoAssuncao changed the title fix: slug_regex when url has different host fix(Gitlab): slug_regex when url has different host Mar 11, 2022
@ashkulz
Copy link
Member

ashkulz commented Apr 4, 2022

Thanks for the contribution, @ArmandoAssuncao! Sorry for the late review.

@ashkulz
Copy link
Member

ashkulz commented Apr 4, 2022

@ArmandoAssuncao can you make a change as per the rubocop failure?

lib/pronto/gitlab.rb:90 W: Style/RegexpLiteral: Use `//` around regular expression.

@ArmandoAssuncao ArmandoAssuncao force-pushed the fix/slug_regex_with_different_host branch from fae60ca to d7cbfe8 Compare April 4, 2022 11:54
@ArmandoAssuncao
Copy link
Contributor Author

@ashkulz fixed

@ashkulz ashkulz merged commit 9143cfe into prontolabs:master Apr 5, 2022
@ashkulz
Copy link
Member

ashkulz commented Apr 5, 2022

Thanks for the contribution!

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.

None yet

2 participants