Skip to content
This repository has been archived by the owner on Mar 9, 2024. It is now read-only.

[FIX] Added .env file to .dockerignore #232

Merged
merged 1 commit into from
Oct 4, 2023
Merged

[FIX] Added .env file to .dockerignore #232

merged 1 commit into from
Oct 4, 2023

Conversation

rmanaem
Copy link
Collaborator

@rmanaem rmanaem commented Oct 4, 2023

Closes #210

Checklist

  • PR has an interpretable title with a prefix ([ENH], [FIX], [REF], [TST], [CI], [MNT], [INF], [MODEL], [DOC]) (see https://neurobagel.org/contributing/pull_requests for more info)
  • PR links to GitHub issue with mention Closes #XXXX
  • Tests pass
  • Checks pass

For new features:

  • Tests have been added

For bug fixes:

  • There is at least one test that would fail under the original bug conditions.

@netlify
Copy link

netlify bot commented Oct 4, 2023

Deploy Preview for neurobagel-query-tool ready!

Name Link
🔨 Latest commit 33a757e
🔍 Latest deploy log https://app.netlify.com/sites/neurobagel-query-tool/deploys/651db13ebf2828000880bfd6
😎 Deploy Preview https://deploy-preview-232--neurobagel-query-tool.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.

@rmanaem rmanaem changed the title Added .env file to .dockerignore [FIX] Added .env file to .dockerignore Oct 4, 2023
Copy link
Contributor

@alyssadai alyssadai 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 to me 🧑‍🍳

@rmanaem rmanaem merged commit b554db8 into main Oct 4, 2023
@rmanaem rmanaem deleted the bug-210 branch October 4, 2023 19:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Exclude the .env file from docker build context
2 participants