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.

Sign up for GitHub

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

Jump to bottom

[Backport standard-runners] Fix the repo name calculation logic in the Updoc CI workflow #197

Merged
merged 1 commit into from
Mar 14, 2024
Merged

[Backport standard-runners] Fix the repo name calculation logic in the Updoc CI workflow #197

merged 1 commit into from
Mar 14, 2024

Conversation

ulucinar
Copy link
Contributor

Backport of #196 to standard-runners.

@ulucinar
Fix the repo name calculation logic in the Updoc CI workflow
f87b8ba 
Signed-off-by: Alper Rifat Ulucinar <[email protected]>
(cherry picked from commit a25ff1a)
@ulucinar ulucinar requested a review from sergenyalcin as a code owner March 14, 2024 16:17
@ulucinar ulucinar changed the base branch from main to standard-runners March 14, 2024 16:18
@ulucinar ulucinar closed this Mar 14, 2024
@ulucinar ulucinar reopened this Mar 14, 2024
sergenyalcin
sergenyalcin approved these changes Mar 14, 2024
View reviewed changes
Copy link
Member

@sergenyalcin sergenyalcin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @ulucinar LGTM. The review of this change was done here.

@ulucinar ulucinar merged commit b5f2870 into upbound:standard-runners Mar 14, 2024
7 checks passed
@ulucinar ulucinar deleted the standard-runners-pr-196 branch March 14, 2024 16:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sergenyalcin sergenyalcin sergenyalcin approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ulucinar @sergenyalcin