Skip to content

feat: 🎸 Docker setup #124

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

Merged
merged 1 commit into from
Feb 24, 2025
Merged

feat: 🎸 Docker setup #124

merged 1 commit into from
Feb 24, 2025

Conversation

yashksaini-coder
Copy link
Owner

Pull Request Template

Summary

This pull request includes several important changes to the project, focusing on Docker integration and build process improvements. The most significant changes are the addition of a .dockerignore file, the creation of a Dockerfile, and modifications to the package.json scripts.

Docker integration:

  • Added a .dockerignore file to exclude unnecessary files and directories from the Docker build context. This includes environment files, version control directories, development files, IDE-specific files, build directories, log files, Docker-specific files, documentation, and test files.
  • Created a Dockerfile to define the Docker image for the application. This includes using the official Node.js image, setting the working directory, copying necessary files, installing dependencies, copying the rest of the application code, exposing the port, and defining the command to run the application.

Build process improvements:

  • Updated the start script in package.json to include the build step before starting the application. This ensures that the Prisma client is generated and the Next.js application is built before it starts.

Type of Change

Please mark [X] for applicable items:

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

Screenshots/Videos (optional)

image

@yashksaini-coder yashksaini-coder added the enhancement New feature or request label Feb 24, 2025
@yashksaini-coder yashksaini-coder self-assigned this Feb 24, 2025
Copy link

vercel bot commented Feb 24, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
leetcode-journal ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 24, 2025 9:45am

Copy link

🚀 Hello @yashksaini-coder!

Thank you for your contribution to the Leetcode Journal project! 🎉
We value your effort and are excited to review your changes.

PR Checklist:

Please ensure your PR adheres to the following checklist:

  • PR description includes a summary and context for the changes.
  • Relevant dependencies have been listed.
  • Testing section is filled out with details on verification.
  • Screenshots/Videos are attached (if applicable).
  • Checklist items are marked as completed.

Review Notifications:

The team will review your PR shortly. If you have any questions, feel free to ask here!
Happy Coding! 🚀

@github-actions github-actions bot added the Under Review The Pull Request is under Review label Feb 24, 2025
@yashksaini-coder yashksaini-coder merged commit 6dd6cb6 into main Feb 24, 2025
4 of 5 checks passed
@yashksaini-coder yashksaini-coder deleted the docker branch February 24, 2025 09:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Under Review The Pull Request is under Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant