[PR #4] [MERGED] Update Dockerfile to fix a few important typos #32

New issue

Closed

opened 2026-05-23 08:39:07 -06:00 by gitea-mirror · 0 comments

gitea-mirror commented

2026-05-23 08:39:07 -06:00

Owner

📋 Pull Request Information

Original PR: https://github.com/appy-one/acebase-server/pull/4
Author: @willrogers007
Created: 4/19/2021
Status: ✅ Merged
Merged: 4/22/2021
Merged by: @appy-one

Base: master ← Head: patch-1

📝 Commits (1)

9416b90 Update Dockerfile to fix a few important typos

📊 Changes

1 file changed (+4 additions, -4 deletions)

View changed files

📝 Dockerfile (+4 -4)

📄 Description

doesn't override shell's PATH to be "." (which was breaking all commands)
points to the correct location of the start.js file (src/start.js)
corrects the suggested "To run" comment to have acebase instead of acebace (with a second 'c')

This dramatically improves the dockerfile's functionality, however it doesn't get the setup to a fully running state for me due to a TypeError at:
/app/node_modules/acebase/src/storage.js:99
TypeError: Cannot read property 'on' of undefined

(will continue chipping away at it)

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/appy-one/acebase-server/pull/4 **Author:** [@willrogers007](https://github.com/willrogers007) **Created:** 4/19/2021 **Status:** ✅ Merged **Merged:** 4/22/2021 **Merged by:** [@appy-one](https://github.com/appy-one) **Base:** `master` ← **Head:** `patch-1` --- ### 📝 Commits (1) - [`9416b90`](https://github.com/appy-one/acebase-server/commit/9416b902f78286f5fc5fff289de6dd62778d6025) Update Dockerfile to fix a few important typos ### 📊 Changes **1 file changed** (+4 additions, -4 deletions) <details> <summary>View changed files</summary> 📝 `Dockerfile` (+4 -4) </details> ### 📄 Description - doesn't override shell's PATH to be "." (which was breaking all commands) - points to the correct location of the start.js file (src/start.js) - corrects the suggested "To run" comment to have acebase instead of acebace (with a second 'c') This dramatically improves the dockerfile's functionality, however it doesn't get the setup to a fully running state for me due to a TypeError at: /app/node_modules/acebase/src/storage.js:99 TypeError: Cannot read property 'on' of undefined (will continue chipping away at it) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>