Merge pull request #1 from metabrainz/master

[FEAT]: DARK MODE THEME ADDED TO BOOKBRAINZ WEBSITE

.gitattributes

@@ -1,15 +1,7 @@
-# Cross-platform line endings configuration
-# Set the default behavior, in case people don't have core.autocrlf set.
-* text=auto
+# all text files follow Unix end-of-line convention
+* text eol=lf
 
-# Explicitly declare text files you want to always be normalized and converted
-# to native line endings on checkout.
-*.sh text
-
-# Declare files that will always have CRLF line endings on checkout.
-# *.sln text eol=crlf
-
-# Denote all files that are truly binary and should not be modified.
+# avoid modifying binary files
 *.png binary
 *.jpg binary
 *.svg binary

.github/PULL_REQUEST_TEMPLATE.md

@@ -1,12 +1,13 @@
 <!--
 Before making a pull request, please:
-1. Read the guidelines for contributing
+1. Read the guidelines for contributing: https://github.com/metabrainz/guidelines/blob/master/GitHub.md
 2. Verify that your changes match our coding style
 3. Fill out the requested information
 -->
 
 ### Problem
-<!-- What are you trying to solve? -->
+<!-- What are you trying to solve?
+Add the JIRA ticket number if you are working on one -->
 
 
 ### Solution

.github/workflows/ci.yml

@@ -60,6 +60,7 @@ jobs:
           POSTGRES_USER: postgres
           POSTGRES_PASSWORD: postgres
           POSTGRES_DB: bookbrainz_test
+          POSTGRES_HOST: localhost
 
       - name: Set up node
         uses: actions/setup-node@v1

.gitignore

@@ -9,6 +9,9 @@
 /config/config.json
 /config/config.local.json
 
+# Directory designated to store local compose overrides and related files
+/local/
+
 # react-hot-loader files
 /static/hot
 *.hot-update.js

.vscode/launch.json

@@ -4,6 +4,25 @@
     // For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
     "version": "0.2.0",
     "configurations": [
+        {
+            "type": "node",
+            "request": "attach",
+            "name": "Docker: Attach to Node",
+            "port": 9229,
+            "localRoot": "${workspaceFolder}/src/server/",
+            "remoteRoot": "/home/bookbrainz/bookbrainz-site/src/server/",
+            "restart": true,
+            "sourceMaps": true,
+            "smartStep": true,
+            "outFiles": [
+                "${workspaceFolder}/**/*.js",
+                "!**/node_modules/**"
+            ],       
+            "skipFiles": [
+            "/home/bookbrainz/bookbrainz-site/node_modules/**/*.js",
+            "<node_internals>/**"
+            ]
+        },
         {
             "type": "node",
             "request": "launch",

CONTRIBUTING.md

@@ -25,6 +25,10 @@ to BookBrainz:
  * Use [JSDoc](http://usejsdoc.org/) for commenting code
  * Use [TypeScript](https://www.typescriptlang.org/) everywhere - static type checking is useful
 
+## Pull requests
+
+Please read carefully the [MetaBrainz Github guidelines](https://github.com/metabrainz/guidelines/blob/master/GitHub.md)
+
 ## Commits
 
 We try to keep our commit messages readable and concise, this is especially important as they can't be rewritten—unlike

Dockerfile

@@ -42,7 +42,7 @@ ARG BB_ROOT=/home/bookbrainz/bookbrainz-site
 WORKDIR $BB_ROOT
 RUN chown bookbrainz:bookbrainz $BB_ROOT
 
-RUN echo $GIT_COMMIT_SHA > .git-version
+ENV GIT_COMMIT_SHA=$GIT_COMMIT_SHA
 
 # Files necessary to complete the JavaScript build
 COPY --chown=bookbrainz scripts/ scripts/