Skip to content

Commit

Permalink
Decoupled linting of hand written code (prettier, eslint, etc) from l…
Browse files Browse the repository at this point in the history
…inting of generated code (escheck). We now run the former during top level test, and the latter only during pre-publish after we've run a top level build.

Upgraded the eslint plugin to match new typescript version, fixed a couple of minor "new" lint errors.
  • Loading branch information
smallsaucepan committed Aug 23, 2024
1 parent 7997ffe commit 8a3974e
Show file tree
Hide file tree
Showing 4 changed files with 152 additions and 155 deletions.
17 changes: 9 additions & 8 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -3,18 +3,19 @@
"funding": "https://opencollective.com/turf",
"scripts": {
"docs": "tsx ./scripts/generate-readmes.ts",
"escheck": "npm-run-all escheck:*",
"escheck:cjs": "es-check es8 packages/*/dist/cjs/index.cjs packages/turf/turf.min.js",
"escheck:esm": "es-check --module es8 packages/*/dist/esm/index.js",
"escheck:web": "es-check es5 packages/turf/turf.min.js",
"lint": "npm-run-all lint:*",
"lint:docs": "documentation lint packages/turf-*/index.js",
"lint:escheck-cjs": "es-check es8 packages/*/dist/cjs/index.cjs packages/turf/turf.min.js",
"lint:escheck-esm": "es-check --module es8 packages/*/dist/esm/index.js",
"lint:escheck-web": "es-check es5 packages/turf/turf.min.js",
"lint:eslint": "eslint packages",
"lint:mrl": "mrl check",
"lint:prettier": "prettier --check .",
"postinstall": "husky install",
"preinstall": "npx only-allow pnpm",
"prepublishOnly": "lerna run build && lerna run --scope @turf/turf cdnBundle",
"test": "lerna run test"
"prepublishOnly": "lerna run build && lerna run --scope @turf/turf cdnBundle && pnpm run escheck",
"test": "pnpm run lint && lerna run test"
},
"lint-staged": {
"package.json": [
Expand All @@ -35,16 +36,16 @@
"@monorepolint/core": "0.5.0-alpha.132",
"@monorepolint/rules": "0.5.0-alpha.132",
"@types/node": "18.11.9",
"@typescript-eslint/eslint-plugin": "^6.10.0",
"@typescript-eslint/parser": "^6.10.0",
"@typescript-eslint/eslint-plugin": "^8.2.0",
"@typescript-eslint/parser": "^8.2.0",
"acorn": "^7.4.1",
"camelcase": "^8.0.0",
"d3-queue": "*",
"decamelize": "^6.0.0",
"dependency-tree": "^11.0.0",
"documentation": "^14.0.3",
"es-check": "^7.1.1",
"eslint": "^8.53.0",
"eslint": "^8.57.0",
"eslint-config-prettier": "^9.0.0",
"eslint-plugin-prettier": "^5.0.1",
"esm": "^3.2.25",
Expand Down
2 changes: 1 addition & 1 deletion packages/turf-polygon-tangents/test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -105,7 +105,7 @@ test("turf-polygon-tangents - Issue #1050", (t) => {
try {
t.assert(polygonTangents(pt, poly));
} catch (e) {
t.skip("issue #1050 failed");
t.skip("issue #1050 failed", e);
}
t.end();
});
2 changes: 1 addition & 1 deletion packages/turf/rollup.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ import terser from "@rollup/plugin-terser";
import { babel } from "@rollup/plugin-babel";
import nodePolyfills from "rollup-plugin-polyfill-node";

const pckg = require("./package.json");
import pckg from "./package.json";
const input = "index.ts";

export default [
Expand Down
Loading

0 comments on commit 8a3974e

Please sign in to comment.