Skip to content

fix: add comprehensive .dockerignore for security and optimization#455

Open
sonusharma6-dsa wants to merge 1 commit into
imDarshanGK:mainfrom
sonusharma6-dsa:main
Open

fix: add comprehensive .dockerignore for security and optimization#455
sonusharma6-dsa wants to merge 1 commit into
imDarshanGK:mainfrom
sonusharma6-dsa:main

Conversation

@sonusharma6-dsa
Copy link
Copy Markdown

Fixes #426

  • Added Python cache patterns (*.pyo, .pytest_cache/)
  • Added environment file patterns (.env.*, .env.local)
  • Added Git patterns (.gitignore)
  • Added frontend dependencies (node_modules/)
  • Added documentation patterns (*.md)
  • Added IDE files (.vscode/, .idea/)
  • Prevents accidental leaking of .env API keys into Docker images
  • Reduces Docker image size
  • Faster Docker build times

Copilot AI review requested due to automatic review settings May 29, 2026 19:45
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@imDarshanGK
Copy link
Copy Markdown
Owner

@sonusharma6-dsa update your branch with the latest main changes

@sonusharma6-dsa
Copy link
Copy Markdown
Author

@sonusharma6-dsa update your branch with the latest main changes

Yes doing it

Signed-off-by: sonusharma6-dsa <ramprashadkumar11@gmail.com>
@sonusharma6-dsa
Copy link
Copy Markdown
Author

Hi @imDarshanGK — I've rebased and updated this PR with the latest upstream main. All tests pass locally and the PR is mergeable now. Could you please take a look and, if everything looks good, provide an approving review so it can be merged? Thanks!

@sonusharma6-dsa
Copy link
Copy Markdown
Author

sonusharma6-dsa commented May 30, 2026

@imDarshanGK, could you please re-request the Copilot PR reviewer from the PR UI so the automated review can re-run? Thanks!

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.

[Security] Missing .dockerignore — sensitive files and cache may be copied into Docker image

3 participants