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

refactor: Keep only the common Ci #1383

Merged
merged 8 commits into from
Feb 16, 2023

Conversation

jrmanes
Copy link
Contributor

@jrmanes jrmanes commented Feb 15, 2023

Overview

Hello team!

In order to keep only the pipeline that we've created for common uses, I've removed the previous one, and this one will be the only one.

I've changed the name to:

docker-build-publish-common.yml -> docker-build-publish.yml

It has been already tested.

Thanks in advance!

Checklist

  • Required CI checks are passing
  • Linked issues closed with keywords

Closes 50%: celestiaorg/devops#61

@jrmanes jrmanes self-assigned this Feb 15, 2023
@jrmanes jrmanes added CI item that directly relates to or modify the CI docker refactor optional label for items that are related to implementation work and do not change functionality labels Feb 15, 2023
@jrmanes jrmanes marked this pull request as ready for review February 15, 2023 07:51
Copy link
Member

@rach-id rach-id left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

Question, the IMAGE_NAME: ${{ github.repository }} is still using the username which can be invalid if contains capital letters. Any plans to fix that?

@smuu
Copy link
Member

smuu commented Feb 15, 2023

LGTM rocket

Question, the IMAGE_NAME: ${{ github.repository }} is still using the username which can be invalid if contains capital letters. Any plans to fix that?

Good point! I used the following in another project.

- name: Set env
  run: echo "IMAGE_NAME=$(echo ${{ github.repository }} | tr '[:upper:]' '[:lower:]')" >> $GITHUB_ENV

@rach-id
Copy link
Member

rach-id commented Feb 15, 2023

I like that solution 👍

@MSevey MSevey requested a review from a team February 16, 2023 07:50
@evan-forbes evan-forbes merged commit d759b54 into celestiaorg:main Feb 16, 2023
rach-id pushed a commit to rach-id/celestia-app that referenced this pull request May 25, 2023
## Overview

Hello team!

In order to keep only the pipeline that we've created for common uses,
I've removed the previous one, and this one will be the only one.

I've changed the name to:
```
docker-build-publish-common.yml -> docker-build-publish.yml
```

*It has been already tested.*

Thanks in advance!

## Checklist

- [x] Required CI checks are passing
- [x] Linked issues closed with keywords

Closes 50%: celestiaorg/devops#61

(cherry picked from commit d759b54)
evan-forbes pushed a commit that referenced this pull request May 26, 2023
## Overview

Hello team!

In order to keep only the pipeline that we've created for common uses,
I've removed the previous one, and this one will be the only one.

I've changed the name to:
```
docker-build-publish-common.yml -> docker-build-publish.yml
```

*It has been already tested.*

Thanks in advance!

## Checklist

- [x] Required CI checks are passing
- [x] Linked issues closed with keywords

Closes 50%: celestiaorg/devops#61

(cherry picked from commit d759b54)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI item that directly relates to or modify the CI refactor optional label for items that are related to implementation work and do not change functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants