open-sauced / intro

Empowering Your Open Source Journey: From First Contribution to Project Leadership
https://opensauced.pizza/learn
Other
497 stars 96 forks source link

Added Dockerfile in directory #86

Closed Adidem23 closed 8 months ago

Adidem23 commented 8 months ago

I have Added Docker File in the repository according to issue #82

I have added docker image named openpizza and we have to give command

Recordings

https://www.loom.com/share/7d30a814149149918f907783dc4a8cd2?sid=687947ac-1622-47fe-90a4-d9d581ef54c8

Added tests?

netlify[bot] commented 8 months ago

Deploy Preview for sauced-intro ready!

Name Link
Latest commit 4978402c33a1e90d45fcb540a79f751bb7de4ef5
Latest deploy log https://app.netlify.com/sites/sauced-intro/deploys/653a7825c2a2d70008ce51ff
Deploy Preview https://deploy-preview-86--sauced-intro.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

adiati98 commented 8 months ago

Hi @Adidem23, Thank you for your patience.

We've talked about this internally. Because this project doesn't have complex setup and installation, and most of content in this repo are in Markdown, we don't think that Docker approach would be helpful here.

That said, we're closing your PR and closes issue #82 . However, we appreciate your ideas. Thank you. :)

Adidem23 commented 8 months ago

Bad 😞 to hear that