Add CI lint and test (#82)

* Add CI

* fmt

Author: kylebarron

.github/workflows/ci.yml

@@ -0,0 +1,48 @@
+name: CI
+
+on:
+  push:
+    branches:
+      - 'main'
+  pull_request:
+
+concurrency:
+  group: ${{ github.workflow }}-${{ github.ref }}
+  cancel-in-progress: true
+
+
+jobs:
+  lint-and-test:
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v3
+
+      - name: Set up Volta
+        uses: volta-cli/action@v4
+
+      - name: Get npm cache directory
+        id: npm-cache-dir
+        shell: bash
+        run: echo "dir=$(npm config get cache)" >> ${GITHUB_OUTPUT}
+
+      - uses: actions/cache@v3
+        id: npm-cache # use this to check for `cache-hit` ==> if: steps.npm-cache.outputs.cache-hit != 'true'
+        with:
+          path: ${{ steps.npm-cache-dir.outputs.dir }}
+          key: ${{ runner.os }}-node-${{ hashFiles('**/package-lock.json') }}
+          restore-keys: |
+            ${{ runner.os }}-node-
+
+      - name: Install
+        run: npm install
+
+      - name: Prettier check
+        run: npm run fmt:check
+
+      - name: Type check
+        run: npm run typecheck
+
+      - name: Test
+        run: npm run test

.prettierrc.json

@@ -0,0 +1,5 @@
+{
+  "tabWidth": 2,
+  "semi": true,
+  "trailingComma": "all"
+}

package-lock.json

@@ -22,13 +22,16 @@
     "default": "./dist/dist.es.mjs"
   },
   "scripts": {
-    "build": "npm run build:tsc && npm run build:rollup",
-    "build:tsc": "tsc",
     "build:rollup": "rollup -c rollup.config.js",
+    "build:tsc": "tsc",
+    "build": "npm run build:tsc && npm run build:rollup",
     "clean": "rimraf dist",
-    "watch": "tsc --watch",
+    "fmt:check": "prettier './src/**/*.ts' --check",
+    "fmt": "prettier './src/**/*.ts' --write",
+    "prepublishOnly": "npm run clean && npm run build",
     "test": "vitest run",
-    "prepublishOnly": "npm run clean && npm run build"
+    "typecheck": "tsc --build",
+    "watch": "tsc --watch"
   },
   "files": [
     "dist/",
@@ -54,6 +57,7 @@
     "@rollup/plugin-terser": "^0.4.3",
     "@rollup/plugin-typescript": "^11.1.2",
     "apache-arrow": "^13.0.0",
+    "prettier": "^3.1.0",
     "rimraf": "^5.0.5",
     "rollup": "^3.26.2",
     "rollup-plugin-dts": "^5.3.0",

package.json

@@ -108,7 +108,7 @@ const defaultProps: DefaultProps<GeoArrowArcLayerProps> = {
 };
 
 export class GeoArrowArcLayer<
-  ExtraProps extends {} = {}
+  ExtraProps extends {} = {},
 > extends CompositeLayer<Required<GeoArrowArcLayerProps> & ExtraProps> {
   static defaultProps = defaultProps;
   static layerName = "GeoArrowArcLayer";

src/arc-layer.ts

@@ -104,7 +104,7 @@ const defaultProps: DefaultProps<GeoArrowColumnLayerProps> = {
  * coordinates.
  */
 export class GeoArrowColumnLayer<
-  ExtraProps extends {} = {}
+  ExtraProps extends {} = {},
 > extends CompositeLayer<Required<GeoArrowColumnLayerProps> & ExtraProps> {
   static defaultProps = defaultProps;
   static layerName = "GeoArrowColumnLayer";
@@ -130,7 +130,7 @@ export class GeoArrowColumnLayer<
   }
 
   _renderLayersPoint(
-    geometryColumn: PointVector
+    geometryColumn: PointVector,
   ): Layer<{}> | LayersList | null {
     const { data: table } = this.props;
 

src/column-layer.ts

@@ -40,7 +40,7 @@ function earcutSinglePolygon(data: PolygonData, geomIndex: number): number[] {
 
   const slicedFlatCoords = flatCoords.values.subarray(
     coordsBegin * dim,
-    coordsEnd * dim
+    coordsEnd * dim,
   );
 
   const initialCoordIndex = ringOffsets[ringBegin];

src/earcut.ts

@@ -56,7 +56,7 @@ const defaultProps: DefaultProps<GeoArrowH3HexagonLayerProps> = {
 };
 
 export class GeoArrowH3HexagonLayer<
-  ExtraProps extends {} = {}
+  ExtraProps extends {} = {},
 > extends CompositeLayer<Required<GeoArrowH3HexagonLayerProps> & ExtraProps> {
   static defaultProps = defaultProps;
   static layerName = "GeoArrowH3HexagonLayer";

src/h3-hexagon-layer.ts

@@ -70,7 +70,7 @@ const defaultProps: DefaultProps<GeoArrowHeatmapLayerProps> = {
 };
 
 export class GeoArrowHeatmapLayer<
-  ExtraProps extends {} = {}
+  ExtraProps extends {} = {},
 > extends CompositeLayer<Required<GeoArrowHeatmapLayerProps> & ExtraProps> {
   static defaultProps = defaultProps;
   static layerName = "GeoArrowHeatmapLayer";
@@ -92,7 +92,7 @@ export class GeoArrowHeatmapLayer<
   }
 
   _renderLayersPoint(
-    geometryColumn: PointVector
+    geometryColumn: PointVector,
   ): Layer<{}> | LayersList | null {
     const { data: table } = this.props;
 

src/heatmap-layer.ts

@@ -98,7 +98,7 @@ const defaultProps: DefaultProps<GeoArrowPathLayerProps> = {
  * Render lists of coordinate points as extruded polylines with mitering.
  */
 export class GeoArrowPathLayer<
-  ExtraProps extends {} = {}
+  ExtraProps extends {} = {},
 > extends CompositeLayer<Required<GeoArrowPathLayerProps> & ExtraProps> {
   static defaultProps = defaultProps;
   static layerName = "GeoArrowPathLayer";
@@ -112,15 +112,15 @@ export class GeoArrowPathLayer<
 
     const lineStringVector = getGeometryVector(
       table,
-      EXTENSION_NAME.LINESTRING
+      EXTENSION_NAME.LINESTRING,
     );
     if (lineStringVector !== null) {
       return this._renderLayersLineString(lineStringVector);
     }
 
     const multiLineStringVector = getGeometryVector(
       table,
-      EXTENSION_NAME.MULTILINESTRING
+      EXTENSION_NAME.MULTILINESTRING,
     );
     if (multiLineStringVector !== null) {
       return this._renderLayersMultiLineString(multiLineStringVector);
@@ -139,7 +139,7 @@ export class GeoArrowPathLayer<
   }
 
   _renderLayersLineString(
-    geometryColumn: LineStringVector
+    geometryColumn: LineStringVector,
   ): Layer<{}> | LayersList | null {
     const { data: table } = this.props;
 
@@ -206,7 +206,7 @@ export class GeoArrowPathLayer<
   }
 
   _renderLayersMultiLineString(
-    geometryColumn: MultiLineStringVector
+    geometryColumn: MultiLineStringVector,
   ): Layer<{}> | LayersList | null {
     const { data: table } = this.props;