Merge pull request #36 from simple-repository/fix/node-deps

Fix npm dependencies

Author: pelson

.github/workflows/python-publish.yml

@@ -24,17 +24,17 @@ jobs:
 
       - uses: actions/setup-node@v4
         with:
-          node-version: 12
+          node-version: 24
 
-      # Install an older Python (needed for the javascript build to succeed)
+      # Install an older Python (needed for the javascript/scss build to succeed)
       - name: Set up temporary Python 3.9
         uses: actions/setup-python@v3
         with:
           python-version: 3.9
 
       - name: Build JS assets
         run: |
-          cd javascript
+          cd web
           npm install --include=dev
           npm run build
 

.github/workflows/python-test.yml

@@ -23,17 +23,17 @@ jobs:
 
     - uses: actions/setup-node@v4
       with:
-        node-version: 12
+        node-version: 24
 
-    # Install an older Python (needed for the javascript build to succeed)
+    # Install an older Python (needed for the javascript/scss build to succeed)
     - name: Set up temporary Python 3.9
       uses: actions/setup-python@v3
       with:
         python-version: 3.9
 
     - name: Build JS assets
       run: |
-        cd javascript
+        cd web
         npm install --include=dev
         npm run build
 

MANIFEST.in

@@ -1,4 +1,4 @@
 # Ensure the sdist doesn't get unnecessary build files.
-recursive-exclude javascript *
+recursive-exclude web *
 recursive-exclude screenshots *
 recursive-exclude .github *

README.md

@@ -59,10 +59,18 @@ versions. There is currently no intelligent cache invalidation for those cases.
 
 ## Development
 
-In order to build the ``simple-repository-browser``, first:
+In order to build the ``simple-repository-browser`` you will need access to npm. If not available on the development host,
+you can do it in a docker container first:
 
 ```bash
-cd javascript
+docker run --rm -v $PWD:/workdir -it node bash
+cd /workdir
+```
+
+Bundle the web app:
+
+```bash
+cd web
 npm install --include=dev
 npm run build
 cd ..