Skip to content

Commit

Permalink
Apply review comments
Browse files Browse the repository at this point in the history
  • Loading branch information
ad-m committed Feb 11, 2021
1 parent cf6bb7e commit f725751
Show file tree
Hide file tree
Showing 5 changed files with 14 additions and 39 deletions.
12 changes: 8 additions & 4 deletions .github/workflows/javascript.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,14 +9,18 @@ jobs:
runs-on: ubuntu-latest
strategy:
matrix:
node: [ '8.x', '10.x', '12.x', '14.x' ]
node: [ '10.x', '12.x', '14.x' ]
steps:
- uses: actions/checkout@v1
- name: Set up Node
uses: actions/setup-node@v1
with:
node-version: '${{ matrix.node }}'
- name: Build
run: make build
- name: Install
run: npm install
- name: Lint via prettier
run: npm run prettier
- name: Lint via eslint
run: npm run eslint
- name: Test
run: make test
run: npm run test
1 change: 0 additions & 1 deletion Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -7,4 +7,3 @@ EXPOSE 8080
VOLUME /data
COPY . .
CMD ["node","bin/s3rver.js", "-p", "8080", "-d", "/data"]

31 changes: 0 additions & 31 deletions Makefile

This file was deleted.

4 changes: 2 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -53,15 +53,15 @@ $ s3rver --help
Build image

```bash
$ make build_docker
$ npm run docker:build
```

You will now have a image `jamhall/s3rver:latest` available.

Executing this command to start server:

```bash
$ make start_docker
$ npm run docker:start
```

Server will listen on port `8080`.
Expand Down
5 changes: 4 additions & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,10 @@
"test": "mocha",
"eslint": "eslint --ignore-path .gitignore .",
"prettier": "prettier --ignore-path .gitignore \"**/*.{js,json,md}\"",
"format": "npm run eslint -- --fix && npm run prettier -- --write"
"format": "npm run eslint -- --fix && npm run prettier -- --write",
"docker:build": "docker build -t jamhall/s3rver:latest .",
"docker:push":"docker build -t jamhall/s3rver:latest .",
"docker:start": "docker run -v $(pwd)/data:/data -p 8080:8080 jamhall/s3rver:latest"
},
"main": "lib/s3rver.js",
"files": [
Expand Down

0 comments on commit f725751

Please sign in to comment.