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

Update codeowners to reflect fork #553

Merged
merged 5 commits into from
Sep 29, 2021
Merged

Conversation

evan-forbes
Copy link
Member

I forgot to change the codeowners in #534, this PR changes that along with updating a few other minor things in the github actions directory

  • update codeowners
  • change docker image name (this won't be used until we setup a docker account)
  • don't bother attempting to generate new docker images for compiling proto-buf
  • updates PR template to what we used before

part of #528

@evan-forbes evan-forbes self-assigned this Sep 28, 2021
@evan-forbes evan-forbes added github_actions Pull requests that update Github_actions code T:chore labels Sep 28, 2021
Copy link
Member

@liamsi liamsi left a comment

Choose a reason for hiding this comment

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

LGTM

@@ -18,7 +18,7 @@ jobs:
- name: Prepare
id: prep
run: |
DOCKER_IMAGE=tendermint/tendermint
DOCKER_IMAGE=celestiaorg/celestia-core
Copy link
Member

Choose a reason for hiding this comment

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

Do we actually push that image to somewhere?

Copy link
Member

Choose a reason for hiding this comment

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

Nope for now afaik
A good finding for the issue #554 🧐

Copy link
Member Author

Choose a reason for hiding this comment

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

we won't be pushing anything unless we create a docker hub account and attach it to the celestiaorg github account. Decided to change the docker image name instead of deleting the file, as that seemed useful

@Bidon15 Bidon15 mentioned this pull request Sep 29, 2021
Copy link
Member

@Bidon15 Bidon15 left a comment

Choose a reason for hiding this comment

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

lgtm 🥳

@evan-forbes evan-forbes merged commit 4a22da5 into master Sep 29, 2021
@evan-forbes evan-forbes deleted the evan/edit-code-owners branch September 29, 2021 13:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
github_actions Pull requests that update Github_actions code T:chore
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants