Skip to content

Commit

Permalink
chore: remove gts and eslint, fix ci
Browse files Browse the repository at this point in the history
  • Loading branch information
alexander-fenster committed Feb 1, 2023
1 parent 7e18c00 commit f5ed199
Show file tree
Hide file tree
Showing 17 changed files with 192 additions and 1,603 deletions.
9 changes: 0 additions & 9 deletions .eslintignore

This file was deleted.

3 changes: 0 additions & 3 deletions .eslintrc.json

This file was deleted.

7 changes: 3 additions & 4 deletions .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -55,15 +55,13 @@ jobs:
chmod +x bazel-bin/protoc_plugin.sh
./gapic-error-conformance -plugin="bazel-bin/protoc_plugin.sh"
- name: Run linting
run: bazel run @nodejs//:npm -- run lint

- name: Prepare baseline artifacts
run: |
mkdir -p ~/artifacts/test-application-runners
tar cfz ~/artifacts/test-protos.tar.gz -C test-fixtures protos
cp bazel-bin/typescript/test/test-application/test-*.js ~/artifacts/test-application-runners/
cp bazel-bin/typescript/test/test-application/test-*.js package.json ~/artifacts/test-application-runners/
cp bazel-testlogs/unit_tests/test.outputs/outputs.zip ~/artifacts/
bazel run -- @pnpm//:pnpm --dir $PWD install
tar cfz ~/artifacts/node_modules.tar.gz node_modules
- name: Save artifacts
Expand Down Expand Up @@ -102,6 +100,7 @@ jobs:
mv ~/artifacts/outputs.zip bazel-testlogs/unit_tests/test.outputs/
tar xzf ~/artifacts/node_modules.tar.gz
mv ~/artifacts/test-application-runners/test-${{ matrix.test }}.js .
mv ~/artifacts/test-application-runners/package.json .
- name: Run ${{ matrix.test }} test application
run: npx mocha test-${{ matrix.test }}.js
Expand Down
6 changes: 0 additions & 6 deletions .prettierignore

This file was deleted.

17 changes: 0 additions & 17 deletions .prettierrc.js

This file was deleted.

6 changes: 1 addition & 5 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -32,9 +32,7 @@
"compile-protos-js": "pbjs -t static-module -o protos/index.js -p node_modules/google-gax/build/protos -p protos google/api/annotations.proto google/api/field_behavior.proto google/api/resource.proto google/api/routing.proto google/longrunning/operations.proto google/protobuf/compiler/plugin.proto service_config.proto snippet_index.proto",
"compile-protos-dts": "pbts protos/index.js -o protos/index.d.ts",
"docker-test": "sh docker/test.sh",
"fix": "gts fix",
"js-test-application": "mocha bazel-bin/typescript/test/test-application/test-js --timeout 600000",
"lint": "gts check",
"prepack": "npm run compile && cd templates/typescript_gapic && rm -f package.json.njk && mv package.json package.json.njk && cd ../.. && mkdir -p build && cp -rf bazel-bin/typescript templates protos build/",
"postpack": "cd templates/typescript_gapic && mv package.json.njk package.json && ln -s package.json package.json.njk",
"test": "bazel test --test_output=errors //:unit_tests",
Expand All @@ -60,13 +58,11 @@
"@types/nunjucks": "^3.2.1",
"@types/object-hash": "^3.0.2",
"@types/yargs": "^17.0.17",
"c8": "^7.12.0",
"espower-typescript": "^10.0.1",
"gts": "^3.1.1",
"mocha": "^10.2.0",
"typescript": "4.9.3"
},
"engines": {
"node": ">=14"
"node": ">=18"
}
}
Loading

0 comments on commit f5ed199

Please sign in to comment.