ci: fail on lint warnings (#522)

* ci: fail lint on warnings
* style: fix linting
* docs: changelog

CHANGELOG.md

@@ -5,6 +5,7 @@ Please add unreleased changes in the following style:
 - PR Title ([#___](https://github.com/maplibre/maplibre-react-native/pull/___))
 ```
 
+- ci: fail on lint warning ([#522](https://github.com/maplibre/maplibre-react-native/pull/522))
 - fix: allow MapView and Images to have no children ([#521](https://github.com/maplibre/maplibre-react-native/pull/521))
 - docs: reformat changelog ([#520](https://github.com/maplibre/maplibre-react-native/pull/520))
 - refactor: rename `RCTMLN` to `MLRN` ([#519](https://github.com/maplibre/maplibre-react-native/pull/519))

package.json

@@ -28,12 +28,12 @@
     "test": "jest",
     "lint": "yarn lint:tsc && yarn lint:eslint",
     "lint:tsc": "tsc --project ./ && tsc --project ./packages/examples",
-    "lint:eslint": "eslint ./",
-    "lint:eslint:fix": "eslint ./ --fix",
+    "lint:eslint": "eslint ./ --max-warnings 0",
+    "lint:eslint:fix": "yarn lint:eslint --fix",
     "plugin:build": "tsc --build ./plugin",
     "plugin:lint": "yarn plugin:lint:tsc && yarn plugin:lint:eslint",
     "plugin:lint:tsc": "tsc --noEmit --project ./plugin",
-    "plugin:lint:eslint": "eslint ./plugin/src/*",
+    "plugin:lint:eslint": "eslint ./plugin/src/* --max-warnings 0",
     "plugin:test": "jest ./plugin",
     "prepack": "bob build && yarn plugin:build"
   },

src/components/ImageSource.tsx

@@ -36,8 +36,7 @@ export interface ImageSourceProps extends BaseProps {
 
 type NativeProps = ImageSourceProps;
 
-const MLRNImageSource =
-  requireNativeComponent<NativeProps>(NATIVE_MODULE_NAME);
+const MLRNImageSource = requireNativeComponent<NativeProps>(NATIVE_MODULE_NAME);
 
 /**
  * ImageSource is a content source that is used for a georeferenced raster image to be shown on the map.

src/components/Light.tsx

@@ -34,9 +34,7 @@ const Light: React.FC<LightProps> = (props: LightProps) => {
     ...props,
   });
 
-  return (
-    <MLRNLight ref={setNativeLayer} testID="mlrnLight" {...baseProps} />
-  );
+  return <MLRNLight ref={setNativeLayer} testID="mlrnLight" {...baseProps} />;
 };
 
 export default Light;

src/components/MarkerView.tsx

@@ -83,7 +83,6 @@ const MarkerView = ({
 
 let lastId = 0;
 
-const MLRNMarkerView =
-  requireNativeComponent<NativeProps>(NATIVE_MODULE_NAME);
+const MLRNMarkerView = requireNativeComponent<NativeProps>(NATIVE_MODULE_NAME);
 
 export default MarkerView;