Skip to content
New issue

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

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

Already on GitHub? # to your account

add extra case for gitlab repo dir name #654

Merged
merged 3 commits into from
Dec 6, 2023
Merged

Conversation

jpetrucciani
Copy link
Contributor

This was a small change we had to make for solving the issue with #651 wherein we see a different name structure for the directory within the tarball that we get from gitlab

@leg100
Copy link
Owner

leg100 commented Dec 5, 2023

@jpetrucciani Thank you. See the review I've made. And please merge in master.

@jpetrucciani
Copy link
Contributor Author

@leg100

I've merged master in! But I'm not currently able to see a review on here - could you link me?

@leg100
Copy link
Owner

leg100 commented Dec 5, 2023

@leg100

I've merged master in! But I'm not currently able to see a review on here - could you link me?

https://github.com/leg100/otf/pull/654/files/d988a1f0da0b4baeb57f3d0628dbdac285646b8b

@jpetrucciani
Copy link
Contributor Author

apologies if i'm missing something - when i click that link, i just see this:

image

@leg100
Copy link
Owner

leg100 commented Dec 5, 2023

apologies if i'm missing something - when i click that link, i just see this:

image

Ah sorry, my bad, my review was still in pending (I've never used this functionality before). You should see it now.

@jpetrucciani
Copy link
Contributor Author

@leg100 no worries! Thank you for your responsiveness!

I've pushed that update

@jpetrucciani jpetrucciani requested a review from leg100 December 5, 2023 22:17
@leg100 leg100 merged commit 5424565 into leg100:master Dec 6, 2023
leg100 pushed a commit that referenced this pull request Dec 7, 2023
🤖 I have created a release *beep* *boop*
---


## [0.2.1](v0.2.0...v0.2.1)
(2023-12-07)


### Bug Fixes

* add extra case for gitlab repo dir name
([#654](#654))
([5424565](5424565))
* organization tokens ([#660](#660))
([be82c55](be82c55))
* various agent pool and job bugs
([#659](#659))
([ed9b1fd](ed9b1fd))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@leg100 leg100 mentioned this pull request Dec 9, 2023
leg100 added a commit that referenced this pull request Dec 12, 2023
This PR builds upon #654 to fix further Gitlab issues, and to bring it
generally into line with the Github support.

Fixes #651
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants