Inital Linter Setup with Actions (#88)

* Inital Linter Setup

* Fixed Linting Errors From Eslint

.github/workflows/linter.yaml

@@ -0,0 +1,25 @@
+name: Linter
+on: 
+  push: 
+    branches: 
+      - main
+
+jobs:
+  eslint:
+    name: Lint Code Base
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout Source Code
+        uses: actions/checkout@v2
+      - name: Set up Node.js
+        uses: actions/setup-node@v3
+        with:
+          node-version: '16'
+      - name: Install dependencies
+        run: |
+          cd client
+          npm install
+      - name: Run eslint
+        run: |
+          cd client
+          npm run lint -- --fix

client/contexts/DarkModeContext.js

@@ -1,4 +1,5 @@
-import React, { createContext, useState, useContext } from 'react';
+import React, { createContext, useState, useContext, useMemo, useCallback } from 'react';
+import PropTypes from 'prop-types';
 
 const DarkModeContext = createContext();
 
@@ -9,11 +10,21 @@ export function useDarkMode() {
 export function DarkModeProvider({ children }) {
   const [isDarkMode, setIsDarkMode] = useState(false);
 
-  const toggleSwitch = () => setIsDarkMode(!isDarkMode);
+  // Wrap toggleSwitch in useCallback to prevent re-creation on every render
+  const toggleSwitch = useCallback(() => {
+    setIsDarkMode((prev) => !prev);
+  }, []);
+
+  // Memoize the context value
+  const value = useMemo(() => ({ isDarkMode, toggleSwitch }), [isDarkMode, toggleSwitch]);
 
   return (
-    <DarkModeContext.Provider value={{ isDarkMode, toggleSwitch }}>
+    <DarkModeContext.Provider value={value}>
       {children}
     </DarkModeContext.Provider>
   );
 }
+
+DarkModeProvider.propTypes = {
+  children: PropTypes.node.isRequired,
+};

client/package.json

@@ -25,6 +25,7 @@
     "expo-notifications": "^0.28.16",
     "expo-sms": "^12.0.1",
     "expo-status-bar": "^1.12.1",
+    "prop-types": "^15.8.1",
     "react": "18.2.0",
     "react-dom": "18.2.0",
     "react-native": "^0.74.5",

client/services/sms.js

@@ -1,3 +1,6 @@
+// Function not used, please finish implementation
+
+/* 
 import * as SMS from 'expo-sms';
 
 const sendAlarmNotification = async (phoneNumber) => {
@@ -6,3 +9,4 @@ const sendAlarmNotification = async (phoneNumber) => {
     await SMS.sendSMSAsync(phoneNumber, 'Alarm missed! Wake up!');
   }
 };
+*/