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

feat: cosmos docker image #5

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

feat: cosmos docker image #5

wants to merge 4 commits into from

Conversation

faddat
Copy link

@faddat faddat commented Sep 8, 2022 •
edited
Loading

This dockerfile & builder:

  • works on arm and amd64
  • uses standard pacman packages
  • can be used to make VM images and raspberry pi images
  • uploads to ghcr daily
  • has buf
  • has protobuf tools
  • has golangci-lint
  • has rust stable for cosmwasm work
  • has all needed database libraries

@faddat faddat requested a review from a team as a code owner September 8, 2022 15:08
@faddat faddat changed the title cosmos docker image feat: cosmos docker image Sep 8, 2022
julienrbrt
julienrbrt approved these changes Sep 8, 2022
View reviewed changes
Copy link
Member

@julienrbrt julienrbrt left a comment •
edited
Loading

Choose a reason for hiding this comment

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

nit: can we beautify that yaml and add a README?

tac0turtle
tac0turtle reviewed Sep 9, 2022
View reviewed changes
.github/workflows/cosmos.yml
on:
workflow_dispatch:
pull_request:
branches: main
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
branches: main

tac0turtle
tac0turtle reviewed Sep 9, 2022
View reviewed changes
.github/workflows/cosmos.yml
push:
branches: main
schedule:
- cron: '0 1 * * *'
Copy link
Member

Choose a reason for hiding this comment

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

why does this need to be built lateset

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tac0turtle tac0turtle tac0turtle left review comments

@julienrbrt julienrbrt julienrbrt 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.
3 participants
@faddat @tac0turtle @julienrbrt