Skip to content

Commit

Permalink
Merge pull request #135 from Financial-Times/fix_build_script
Browse files Browse the repository at this point in the history
ACC-2796: fix build script
  • Loading branch information
danieleruiz authored Nov 17, 2023
2 parents 39c2f20 + 95f93d8 commit 0fdd790
Show file tree
Hide file tree
Showing 7 changed files with 10,329 additions and 12,253 deletions.
3 changes: 2 additions & 1 deletion .eslintrc.js
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
module.exports = {
extends: ['@financial-times/eslint-config-next']
extends: ['@financial-times/eslint-config-next'],
ignorePatterns: ['dist/'],
};
2 changes: 1 addition & 1 deletion .github/settings.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ branches:
required_status_checks:
strict: true
contexts:
- "ci/circleci: build"
- "ci/circleci: build-node"
enforce_admins: false
restrictions:
users: []
Expand Down
5 changes: 4 additions & 1 deletion .toolkitrc.yml
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
plugins:
- "@dotcom-tool-kit/webpack"
- "@dotcom-tool-kit/component"
- "@dotcom-tool-kit/jest"
- "@dotcom-tool-kit/eslint"
Expand All @@ -7,4 +8,6 @@ hooks:
- Eslint
- JestLocal
test:ci:
- JestCI
- JestCI
build:local:
- WebpackDevelopment
4 changes: 2 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -27,13 +27,13 @@ If your app is failing locally as if you need the syndication API installed loca
```sh
$ git clone git@github.com:Financial-Times/n-syndication.git
$ cd n-syndication
$ make install
$ npm install
```

Run the (incomplete) test suite with

```sh
$ make test
$ npm test
```

Verify that the Sass can compile by running
Expand Down
Loading

0 comments on commit 0fdd790

Please sign in to comment.