Merge pull request #303 from opentripplanner/enforce-codespell

Correct spelling errors and enforce codespell

.github/workflows/codespell.yml

@@ -12,5 +12,4 @@ jobs:
       - uses: codespell-project/actions-codespell@master
         with:
           check_filenames: true
-          only_warn: 1
           skip: ./.git,yarn.lock

lib/actions/api.js

@@ -350,7 +350,7 @@ export function carRentalQuery (params) {
   return createQueryAction('car_rental', carRentalResponse, carRentalError)
 }
 
-// Vehicle rental locations lookup query. For now, there are 3 seperate
+// Vehicle rental locations lookup query. For now, there are 3 separate
 // "vehicle" rental endpoints - 1 for cars, 1 for bicycle rentals and another
 // for micromobility. In the future, the hope is to consolidate these 3
 // endpoints into one.

lib/actions/user.js

@@ -62,7 +62,7 @@ export function fetchAuth0Token (auth0) {
 
       dispatch(setCurrentUser({ accessToken }))
     } catch (error) {
-      // TODO: improve UI if there is an errror.
+      // TODO: improve UI if there is an error.
       alert('Error obtaining an authorization token.')
     }
   }
@@ -257,7 +257,7 @@ export function requestPhoneVerificationSms (newPhoneNumber) {
     const now = new Date()
 
     // Request a new verification code if we are requesting a different number.
-    // or enough time has ellapsed since the last request (1 minute?).
+    // or enough time has elapsed since the last request (1 minute?).
     // TODO: Should throttling be handled in the middleware?
     if (number !== newPhoneNumber || (now - timestamp) >= 60000) {
       const { accessToken, apiBaseUrl, apiKey, loggedInUser } = getMiddlewareVariables(state)

lib/components/user/notification-prefs-pane.js

@@ -34,7 +34,7 @@ const allowedNotificationChannels = [
 ]
 
 // Styles
-// HACK: Preverve container height.
+// HACK: Preserve container height.
 const Details = styled.div`
   min-height: 150px;
   margin-bottom: 15px;
@@ -132,7 +132,7 @@ const NotificationPrefsPane = ({
                 bsStyle={notificationChannel === type ? 'primary' : 'default'}
                 key={index}
                 // onBlur and onChange have to be set on individual controls instead of the control group
-                // in order for Formik to correcly process the changes.
+                // in order for Formik to correctly process the changes.
                 onBlur={handleBlur}
                 onChange={handleChange}
                 value={type}

lib/index.js

@@ -90,7 +90,7 @@ export {
   TripDetails,
   TripTools,
 
-  // mobile compoments
+  // mobile components
   MobileMain,
 
   // viewer components

lib/reducers/create-otp-reducer.js

@@ -27,10 +27,10 @@ const MAX_RECENT_STORAGE = 5
  * Validates the initial state of the store. This is intended to mainly catch
  * configuration issues since a manually edited config file is loaded into the
  * initial state.
- * TODO: mabye it's a better idea to move all of this to a script that can do
+ * TODO: maybe it's a better idea to move all of this to a script that can do
  *  JSON Schema validation and other stuff.
  */
-function validateInitalState (initialState) {
+function validateInitialState (initialState) {
   const {config} = initialState
 
   const errors = []
@@ -237,8 +237,8 @@ export function getInitialState (userDefinedConfig, initialQuery) {
 
 function createOtpReducer (config, initialQuery) {
   const initialState = getInitialState(config, initialQuery)
-  // validate the inital state
-  validateInitalState(initialState)
+  // validate the initial state
+  validateInitialState(initialState)
 
   return (state = initialState, action) => {
     const searchId = action.payload && action.payload.searchId
@@ -560,7 +560,7 @@ function createOtpReducer (config, initialQuery) {
         if (duplicateIndex !== -1) searches[duplicateIndex] = action.payload
         else searches.unshift(action.payload)
         const sortedSearches = searches.sort((a, b) => b.timestamp - a.timestamp)
-        // Ensure recent searches do not extend beyong MAX_RECENT_STORAGE
+        // Ensure recent searches do not extend beyond MAX_RECENT_STORAGE
         if (sortedSearches.length >= MAX_RECENT_STORAGE) {
           sortedSearches.splice(MAX_RECENT_STORAGE)
         }