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

feat: made the Docker file Multistaged #3237

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

hkv24
Copy link
Contributor

@hkv24 hkv24 commented Sep 15, 2024

Description
The existing Dockerfile was good for development environment. But in order to build it fast for production environment I made 2 files Dockerfile.dev and Dockerfile.prod .
The production file is taking nearly half the time of development file to build.
I also updated the readme.md file.

Resolves #3223

Copy link

netlify bot commented Sep 15, 2024

Deploy Preview for asyncapi-website ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 0bf328f
🔍 Latest deploy log https://app.netlify.com/sites/asyncapi-website/deploys/66e70af4e8aa74000815cf1d
😎 Deploy Preview https://deploy-preview-3237--asyncapi-website.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.

@hkv24 hkv24 changed the title feat: made the Docker file Multistaged (Fixes #3223) feat: made the Docker file Multistaged Sep 15, 2024
@asyncapi-bot
Copy link
Contributor

⚡️ Lighthouse report for the changes in this PR:

Category Score
🔴 Performance 48
🟢 Accessibility 98
🟢 Best practices 92
🟢 SEO 100
🔴 PWA 33

Lighthouse ran on https://deploy-preview-3237--asyncapi-website.netlify.app/

Copy link
Member

@sambhavgupta0705 sambhavgupta0705 left a comment

Choose a reason for hiding this comment

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

left some comments

ENV NODE_ENV=development

CMD ["npm", "run", "dev"]

# Build command -> docker build -f Dockerfile.dev --build-arg NODE_ENV=development -t app-dev .
Copy link
Member

Choose a reason for hiding this comment

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

What does this line of code signify?

@@ -123,14 +123,25 @@ Generated files of the storybook go to the `storybook-static` folder.
After cloning repository to your local, perform the following steps from the root of the repository.

#### Steps:
1. Build the Docker image:
1. Development Setup with Docker:
1. Build the Docker image:
Copy link
Member

Choose a reason for hiding this comment

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

This is not rendering in a proper way
Check the preview in online md file convertor

@@ -0,0 +1,31 @@
FROM node:18-alpine AS build
Copy link
Member

Choose a reason for hiding this comment

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

Please provide the link for this code file from where you have taken inspiration

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.

[FEATURE] For making current docker file a multistaged one.
3 participants