Skip to content

Commit

Permalink
chore: clean-ups [#294](#294) (#294)
Browse files Browse the repository at this point in the history
* Remove use of `eslint` completely (all source code is TypeScript so `tslint` is enough)
* Remove `Dockerfile`, `run_tests.sh` and `.dockerignore`
* Move `bench` to TypeScript
* Remove un-used `bench/dataset/` folder
* Make sure that all sub-packages can be installed and built independently (fix missing deps)
  • Loading branch information
remusao authored Aug 21, 2019
1 parent bf7f709 commit 9b084ac
Show file tree
Hide file tree
Showing 31 changed files with 329 additions and 106,197 deletions.
3 changes: 0 additions & 3 deletions .dockerignore

This file was deleted.

12 changes: 0 additions & 12 deletions .eslintrc

This file was deleted.

7 changes: 4 additions & 3 deletions .metalint.json
Original file line number Diff line number Diff line change
Expand Up @@ -20,9 +20,10 @@
"./packages/adblocker-benchmarks/blockers/cliqzCompression.js",
"./packages/adblocker-benchmarks/create_dataset.js",
"./packages/adblocker-benchmarks/run.js",
"./bench/run_benchmark.js",
"./bench/micro.js",
"./bench/utils.js"
"./bench/run_benchmark.ts",
"./bench/micro.ts",
"./bench/utils.ts",
"./bench/requests.ts"
],
"exclude": [
"./packages/adblocker/src/punycode.ts",
Expand Down
6 changes: 5 additions & 1 deletion .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,11 @@ before_install:
cache: yarn

script:
- ./run_tests.sh
- yarn bootstrap
- yarn lint
- yarn build
- yarn bundle
- yarn test

# TODO - migrate to lerna
# deploy:
Expand Down
6 changes: 6 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,12 @@

*not released*

* chore: clean-ups [#294](https://github.com/cliqz-oss/adblocker/pull/294)
* Remove use of `eslint` completely (all source code is TypeScript so `tslint` is enough)
* Remove `Dockerfile`, `run_tests.sh`
* Move `bench` to TypeScript
* Remove un-used `bench/dataset/` folder
* Make sure that all sub-packages can be installed and built independently (fix missing deps)
* enable @cliqz/metalint for repository linting [#255](https://github.com/cliqz-oss/adblocker/pull/255)

## 0.14.0
Expand Down
6 changes: 0 additions & 6 deletions Dockerfile

This file was deleted.

2 changes: 1 addition & 1 deletion bench/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -2,4 +2,4 @@
.PHONY: all

all:
NODE_ENV=production node --expose-gc run_benchmark.js
NODE_ENV=production node -r ts-node/register --expose-gc run_benchmark.ts
5 changes: 0 additions & 5 deletions bench/dataset/Makefile

This file was deleted.

55 changes: 0 additions & 55 deletions bench/dataset/check_mobile_lists.js

This file was deleted.

143 changes: 0 additions & 143 deletions bench/dataset/create_dataset.js

This file was deleted.

Loading

0 comments on commit 9b084ac

Please sign in to comment.