feat(docker): install build dependencies when run

Author: marcus-sa

CMakeLists.txt

@@ -1,6 +1,8 @@
 cmake_minimum_required(VERSION 2.8)
 
-set (CMAKE_CXX_STANDARD 11)
+set(CMAKE_CXX_STANDARD 11)
+set(CMAKE_EXPORT_COMPILE_COMMANDS ON)
+set(CMAKE_CXX_CLANG_TIDY clang-tidy -checks=-*,readability-*)
 
 project(NgTest)
 

Dockerfile

@@ -0,0 +1,12 @@
+FROM ngqt/qode
+WORKDIR /ng-test
+
+# Install required tools
+# hadolint ignore=DL3008,DL3007
+RUN apt-get update && apt-get install -y --no-install-recommends \
+	cmake \
+	clang \
+	clang-tidy \
+	clang-format
+
+ENTRYPOINT yarn install && /bin/bash

README.md

@@ -4,21 +4,14 @@ A test library for NodeGUI widgets
 ### Development
 The recommended way of getting started with contributing is using [Docker](https://docker.com)
 
-> If you already have Qt5 and Node.js installed on your system, and don't want to use [Docker](https://docker.com), go over to the [Docker image for Qode](https://github.com/ng-qt/qode-docker-image#environment) and set the same environment variables.
+> If you already have QMake, Qt5, Node.js and CLang & friends installed on your system, and don't want to use [Docker](https://docker.com), go over to the [Docker image for Qode](https://github.com/ng-qt/qode-docker-image#environment) and set the same environment variables.
 
-Run the following bash script to start the **ng-test** development environment for [Docker](https://docker.com)
+Run the following command to start the **ng-test** development environment for [Docker](https://docker.com) and install the required build dependencies
 > The project context will be shared with Docker, which means any changes made on your system, or in the container will be reflected to one another.
 ```
 $ ./docker-dev.sh
 ```
 
-Afterwards install the Node dependencies using Yarn
-```
-$ yarn install
-```
-
-
-
 #### Requirements
 * [Docker](https://docker.com)
 * [Qt5](https://doc.qt.io/qt-5/qt5-intro.html)

docker-dev.sh

@@ -1,13 +1,14 @@
 # For local development
 NAME=ng_test
 WORKDIR=/ng-test
-IMAGE=ngqt/qode
+FILE=Dockerfile
 CONTEXT=$(pwd)
 
 # If container already exists, exec into it, otherwise run it
 if docker start $NAME &> /dev/null
 then
   docker exec -w $WORKDIR -it $NAME bash
 else
-  docker run --name $NAME -v "$CONTEXT":$WORKDIR -w $WORKDIR -ti $IMAGE bash
+	docker build -f $FILE -t $NAME .
+  docker run --name $NAME -v "$CONTEXT":$WORKDIR -w $WORKDIR -it $NAME bash
 fi

index.js

@@ -1 +1 @@
-module.exports = require('./build/Release/ngtest.node');
+module.exports = require('./build/Release/NgTest.node');

yarn.lock

@@ -866,7 +866,7 @@ clone-response@^1.0.2:
   dependencies:
     mimic-response "^1.0.0"
 
-cmake-js@^5.3.2:
+[email protected]:
   version "5.3.2"
   resolved "https://registry.yarnpkg.com/cmake-js/-/cmake-js-5.3.2.tgz#822e43151ae39aaf591c327912a786bcc63abcd2"
   integrity sha512-OsRQ3ZgoV3nJ+3bZr9MEvSuhGS7xc3LgmPZ2gCXlLqF5Wxi6mUQNK70dm/2UUf4/QNBssdYWB5Y7BSRX9FFUrg==