Fix lint errors with custom events branch (#52)

* Address eslint errors with custom events branch

* Make CI workflow run regardless of target branch

.github/workflows/quality.yml

@@ -1,8 +1,6 @@
 name: ci
 on:
   pull_request:
-    branches:
-      - main
   push:
     branches:
       - main

packages/web/src/generic.test.ts

@@ -98,6 +98,7 @@ describe('track custom events', () => {
           // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
           nested: {
             object: '',
+            // eslint-disable-next-line @typescript-eslint/no-explicit-any
           } as any,
         });
 
@@ -173,6 +174,7 @@ describe('track custom events', () => {
           // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
           nested: {
             object: '',
+            // eslint-disable-next-line @typescript-eslint/no-explicit-any
           } as any,
         });
 

packages/web/src/generic.ts

@@ -1,4 +1,4 @@
-import { name, version } from '../package.json';
+import { name as packageName, version } from '../package.json';
 import { initQueue } from './queue';
 import type { AllowedPropertyValues, AnalyticsProps } from './types';
 import {
@@ -9,11 +9,11 @@ import {
   isProduction,
 } from './utils';
 
-export const inject = (
+export function inject(
   props: AnalyticsProps = {
     debug: true,
   },
-): void => {
+): void {
   if (!isBrowser()) return;
 
   setMode(props.mode);
@@ -33,20 +33,20 @@ export const inject = (
   const script = document.createElement('script');
   script.src = src;
   script.defer = true;
-  script.setAttribute('data-sdkn', name);
+  script.setAttribute('data-sdkn', packageName);
   script.setAttribute('data-sdkv', version);
 
   if (isDevelopment() && props.debug === false) {
     script.setAttribute('data-debug', 'false');
   }
 
   document.head.appendChild(script);
-};
+}
 
-export const track = (
+export function track(
   name: string,
   properties?: Record<string, AllowedPropertyValues>,
-): void => {
+): void {
   if (!isBrowser()) {
     // eslint-disable-next-line no-console
     console.warn(
@@ -75,7 +75,7 @@ export const track = (
       console.error(err);
     }
   }
-};
+}
 
 // eslint-disable-next-line import/no-default-export
 export default {

packages/web/src/react.test.tsx

@@ -104,6 +104,7 @@ describe('<Analytics />', () => {
           // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
           nested: {
             object: '',
+            // eslint-disable-next-line @typescript-eslint/no-explicit-any
           } as any,
         });
 

packages/web/src/utils.ts

@@ -34,8 +34,12 @@ export function isDevelopment(): boolean {
   return getMode() === 'development';
 }
 
-const removeKey = (key: string, { [key]: _, ...rest }): Record<string, any> =>
-  rest;
+function removeKey(
+  key: string,
+  { [key]: _, ...rest },
+): Record<string, unknown> {
+  return rest;
+}
 
 export function parseProperties(
   properties: Record<string, unknown>,