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

Namadillo: Creating Docker setup #927

Merged
merged 6 commits into from
Jul 15, 2024
Merged

Conversation

pedrorezende
Copy link
Collaborator

This PR introduces the first version of a Dockerfile for Namadillo

Closes #926

Copy link
Contributor

github-actions bot commented Jul 12, 2024

@pedrorezende pedrorezende force-pushed the feat/namadillo/dockerfile branch from 35c12dd to 1ddea3f Compare July 12, 2024 06:47
@github-actions github-actions bot temporarily deployed to pull request July 12, 2024 06:52 Inactive
@pedrorezende pedrorezende force-pushed the feat/namadillo/dockerfile branch from 1ddea3f to d39abbf Compare July 12, 2024 06:53
@github-actions github-actions bot temporarily deployed to pull request July 12, 2024 06:57 Inactive
@pedrorezende pedrorezende force-pushed the feat/namadillo/dockerfile branch from d39abbf to ec5e0a8 Compare July 12, 2024 06:58
@github-actions github-actions bot temporarily deployed to pull request July 12, 2024 07:03 Inactive
@@ -22,6 +22,7 @@
"framer-motion": "^11.0.28",
"idb-keyval": "^6.2.1",
"invariant": "^2.2.4",
"io-ts": "^2.2.21",
Copy link
Collaborator

Choose a reason for hiding this comment

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

@pedrorezende Seems that all of the CI is failing due to this. Maybe we just need to update the yarn.lock file?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Thanks @jurevans I missed the yarn.lock on my commit!

@github-actions github-actions bot temporarily deployed to pull request July 12, 2024 18:59 Inactive
@pedrorezende pedrorezende requested a review from jurevans July 12, 2024 19:19
@Fraccaman
Copy link
Member

@pedrorezende thanks, lgtm!

Copy link
Contributor

@emccorson emccorson left a comment

Choose a reason for hiding this comment

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

Looks good! The only question I have is about how this will be used. If we get bug reports from the GitHub releases built through the CI, it should be easy to track down which version is the problem, but I'm not sure whether that will be the case for Docker builds from main. But I don't know, maybe it's not worth worrying about.

@pedrorezende
Copy link
Collaborator Author

pedrorezende commented Jul 15, 2024

I think the idea is to have a different option to host Namadillo, once Docker is widely used. Also, the CI is generating the build, so we can do whatever we want with that package. On the other hand, the Docker image configures everything we need.

@pedrorezende pedrorezende merged commit 9d78af1 into main Jul 15, 2024
8 checks passed
@mateuszjasiuk mateuszjasiuk deleted the feat/namadillo/dockerfile branch September 13, 2024 09:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Namadillo: Create Dockerfile
4 participants