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 b138e28
Show file tree
Hide file tree
Showing 5 changed files with 13 additions and 38 deletions.
10 changes: 7 additions & 3 deletions .github/workflows/javascript.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,11 @@ jobs:
uses: actions/setup-node@v1
with:
node-version: '${{ matrix.node }}'
- name: Build
run: make build
- name: Install
run: npm install
- name: Lint via prettier check
run: npm run prettier-check
- 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 b138e28

Please sign in to comment.