Skip to content

fix(release): fix Dockerfile#460

Merged
bagelbits merged 3 commits intomainfrom
bagelbits.fix-docker
Apr 1, 2022
Merged

fix(release): fix Dockerfile#460
bagelbits merged 3 commits intomainfrom
bagelbits.fix-docker

Conversation

@bagelbits
Copy link
Copy Markdown
Collaborator

What does this do?

A few things:

  • Adds .dockerignore file (I might just symlink this to .gitignore later if I can)
  • Fixes COPY to --chown=mongodb:mongodb
  • Cleans up Dockerfile so we cache the npm install before we copy the rest of the code.
  • Ran npm update

How was it tested?

I built the image locally.

Is there a Github issue this is resolving?

Nope

Did you update the docs in the API? Please link an associated PR if applicable.

No

Here's a fun image for your troubles

image

@bagelbits bagelbits merged commit f79bf4b into main Apr 1, 2022
@bagelbits bagelbits deleted the bagelbits.fix-docker branch April 1, 2022 01:46
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 1, 2022

🎉 This PR is included in version 1.1.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant