Skip to content

Commit

Permalink
chore: remove --ignore-scripts flag
Browse files Browse the repository at this point in the history
  • Loading branch information
nobkd committed Oct 3, 2023
1 parent d7a00b1 commit 8bba9d6
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 11 deletions.
10 changes: 4 additions & 6 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -29,10 +29,9 @@ jobs:
with:
path: ~/.npm
key: ${{ runner.os }}-node-${{ hashFiles('**/package-lock.json') }}
restore-keys: |
${{ runner.os }}-node-
restore-keys: ${{ runner.os }}-node-

- run: npm ci --ignore-scripts
- run: npm ci

- name: Lint
run: npm run format
Expand All @@ -53,10 +52,9 @@ jobs:
with:
path: ~/.npm
key: ${{ runner.os }}-node-${{ hashFiles('**/package-lock.json') }}
restore-keys: |
${{ runner.os }}-node-
restore-keys: ${{ runner.os }}-node-

- run: npm ci --ignore-scripts
- run: npm ci

- name: Test
run: npm run test:cov
5 changes: 2 additions & 3 deletions .github/workflows/publish.yml
Original file line number Diff line number Diff line change
Expand Up @@ -25,10 +25,9 @@ jobs:
with:
path: ~/.npm
key: ${{ runner.os }}-node-${{ hashFiles('**/package-lock.json') }}
restore-keys: |
${{ runner.os }}-node-
restore-keys: ${{ runner.os }}-node-

- run: npm ci --ignore-scripts
- run: npm ci

- name: Build
run: npm run build
Expand Down
4 changes: 2 additions & 2 deletions AMO-README.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,13 +2,13 @@

## Prequisites

- Node
- Node (18)
- npm

## Installation

```shell
npm ci --ignore-scripts
npm ci
```

This uses the `package-lock.json` file for installation to avoid package changes and have reproducible builds.
Expand Down

0 comments on commit 8bba9d6

Please sign in to comment.