refactor: remove eslint-disable (#4817)

jsjoeio · web-flow · commit 34268e6deea7 · 2022-10-31T22:04:51.000Z
diff --git a/site/src/components/Timeline/Timeline.tsx b/site/src/components/Timeline/Timeline.tsx
@@ -12,8 +12,7 @@ const groupByDate = <TData,>(
   items.forEach((item) => {
     const dateKey = getDate(item).toDateString()
 
-    // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition -- Is not a guarantee a value is defined when access it dynamically
-    if (itemsByDate[dateKey]) {
+    if (dateKey in itemsByDate) {
       itemsByDate[dateKey].push(item)
     } else {
       itemsByDate[dateKey] = [item]
