This is an automated email from the ASF dual-hosted git repository.
alamb pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/datafusion.git
The following commit(s) were added to refs/heads/main by this push:
new c7d5904f01 Add `prettier` to the devcontainer (GitHub codespaces)
(#17019)
c7d5904f01 is described below
commit c7d5904f010235b96e99fbb2f8ea82ee03bbceb3
Author: Andrew Lamb <and...@nerdnetworks.org>
AuthorDate: Mon Aug 11 04:25:49 2025 -0700
Add `prettier` to the devcontainer (GitHub codespaces) (#17019)
* Add nodejs and npm to dec container
* Add prettier
* correct version
* fix
* more hacking
* update npm
---
.devcontainer/Dockerfile | 8 ++++++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/.devcontainer/Dockerfile b/.devcontainer/Dockerfile
index 51111543b4..49aacd118e 100644
--- a/.devcontainer/Dockerfile
+++ b/.devcontainer/Dockerfile
@@ -4,8 +4,12 @@ RUN apt-get update && export DEBIAN_FRONTEND=noninteractive \
# Remove imagemagick due to
https://security-tracker.debian.org/tracker/CVE-2019-10131
&& apt-get purge -y imagemagick imagemagick-6-common
-# Add protoc
+# setup the containers WORKDIR so npm install works
+#
https://stackoverflow.com/questions/57534295/npm-err-tracker-idealtree-already-exists-while-creating-the-docker-image-for
+WORKDIR /root
+
+# Add protoc, npm, prettier
#
https://datafusion.apache.org/contributor-guide/development_environment.html#protoc-installation
RUN apt-get update \
- && apt-get install -y --no-install-recommends protobuf-compiler
libprotobuf-dev \
+ && apt-get install -y --no-install-recommends protobuf-compiler
libprotobuf-dev npm nodejs\
&& rm -rf /var/lib/apt/lists/*
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@datafusion.apache.org
For additional commands, e-mail: commits-h...@datafusion.apache.org
