I like it that way

Author: mikehquan19

parser/cometCalendarParser.go

@@ -259,10 +259,13 @@ func getLocationAbbreviations(inDir string) (map[string]string, []string) {
 			scrapers.ScrapeMapLocations(inDir)
 			time.Sleep(2 * time.Second)
 			ParseMapLocations(inDir, inDir)
-			time.Sleep(2 * time.Second) // Probably a good idea to let it sleep before moving on?
+			time.Sleep(2 * time.Second)
 
-			// If fail to get the locations again, not because unscraped
-			mapFile, _ = os.ReadFile(inDir + "/mapLocations.json")
+			// If fail to get the locations again, it's not because location is unscraped
+			mapFile, err = os.ReadFile(inDir + "/mapLocations.json")
+			if err != nil {
+				panic(err)
+			}
 		} else {
 			panic(err)
 		}
@@ -280,13 +283,13 @@ func getLocationAbbreviations(inDir string) (map[string]string, []string) {
 		// Trim the following acronym in the name
 		trimmedName := strings.Split(*location.Name, " (")[0]
 		// Fallback on the locations that have no acronyms
-		acronym := ""
+		abbreviation := ""
 		if location.Acronym != nil {
-			acronym = *location.Acronym
+			abbreviation = *location.Acronym
 		}
 
-		buildingsAbbreviations[trimmedName] = acronym
-		validAbbreviations = append(validAbbreviations, acronym)
+		buildingsAbbreviations[trimmedName] = abbreviation
+		validAbbreviations = append(validAbbreviations, abbreviation)
 	}
 
 	return buildingsAbbreviations, validAbbreviations

scrapers/cometCalendar.go

@@ -131,8 +131,7 @@ func callAndUnmarshal(client *http.Client, page int, data *APICalendarResponse)
 // getTime parses the start and end time of the event
 func getTime(event RawEvent) (time.Time, time.Time) {
 	instance := convert[map[string]any](
-		convert[map[string]any](
-			convert[[]any](event.Event["event_instances"])[0])["event_instance"])
+		convert[map[string]any](convert[[]any](event.Event["event_instances"])[0])["event_instance"])
 
 	// Converts RFC3339 timestamp string to time.Time
 	startTime, err := time.Parse(time.RFC3339, convert[string](instance["start"]))
@@ -164,28 +163,28 @@ func getEventLocation(event RawEvent) string {
 
 // getFilters parses the types, topics, and target audiences
 func getFilters(event RawEvent) ([]string, []string, []string) {
-	types := []string{}
-	audiences := []string{}
-	topics := []string{}
+	eventTypes := []string{}
+	targetAudiences := []string{}
+	eventTopics := []string{}
 
 	filters := convert[map[string]any](event.Event["filters"])
 
 	rawTypes := convert[[]any](filters["event_types"])
 	for _, rawType := range rawTypes {
-		types = append(types, convert[string](convert[map[string]any](rawType)["name"]))
+		eventTypes = append(eventTypes, convert[string](convert[map[string]any](rawType)["name"]))
 	}
 
 	rawAudiences := convert[[]any](filters["event_target_audience"])
 	for _, audience := range rawAudiences {
-		audiences = append(audiences, convert[string](convert[map[string]any](audience)["name"]))
+		targetAudiences = append(targetAudiences, convert[string](convert[map[string]any](audience)["name"]))
 	}
 
 	rawTopics := convert[[]any](filters["event_topic"])
 	for _, topic := range rawTopics {
-		topics = append(topics, convert[string](convert[map[string]any](topic)["name"]))
+		eventTopics = append(eventTopics, convert[string](convert[map[string]any](topic)["name"]))
 	}
 
-	return types, audiences, topics
+	return eventTypes, targetAudiences, eventTopics
 }
 
 // getDepartmentsAndTags parses the departments, and tags