Skip to content

Conversation

apricote
Copy link
Owner

@apricote apricote commented Sep 8, 2024

Turns out that all we need is the path, and not the project id. The path is way more user friendly, and we can easily get it from a CI variable or combine it from the namespace & project name.

As GitLab support is unreleased, I will remove this commit from the Changelog:

Turns out that all we need is the path, and not the project id. The path
is way more user friendly, and we can easily get it from a CI variable
or combine it from the namespace & project name.
@apricote apricote added the bug Something isn't working label Sep 8, 2024
@apricote apricote added this to the v1.0 milestone Sep 8, 2024
@apricote apricote merged commit ee83cec into main Sep 8, 2024
4 checks passed
@apricote apricote deleted the gitlab-path branch September 8, 2024 19:07
@codecov-commenter
Copy link

Codecov Report

Attention: Patch coverage is 0% with 20 lines in your changes missing coverage. Please review.

Project coverage is 22.94%. Comparing base (634eac3) to head (a908009).

Files with missing lines Patch % Lines
internal/forge/gitlab/gitlab.go 0.00% 19 Missing ⚠️
cmd/rp/cmd/run.go 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #54      +/-   ##
==========================================
+ Coverage   22.89%   22.94%   +0.05%     
==========================================
  Files          21       21              
  Lines        1293     1290       -3     
==========================================
  Hits          296      296              
+ Misses        979      976       -3     
  Partials       18       18              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@apricote apricote mentioned this pull request Sep 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants