Be more strict with publishing dates and skip feed items that don't have any

src/main.rs

@@ -34,7 +34,7 @@ struct Cli {
 struct FeedItem {
     link: String,
     title: String,
-    time: Option<chrono::DateTime<chrono::Utc>>,
+    time: chrono::DateTime<chrono::Utc>,
 }
 
 fn fetch_feed(url: impl IntoUrl) -> Result<Feed> {
@@ -98,8 +98,9 @@ fn make_feed_item(entry: Entry) -> Result<FeedItem> {
             .content
             .clone(),
         time: match entry.published {
-            Some(time) => Some(time),
-            None => entry.updated,
+            Some(time) => time,
+            None => entry.updated
+                .context("Unable to retrieve publishing time from feed entry")?,
         },
     })
 }
@@ -134,7 +135,7 @@ fn main() -> Result<()> {
         }
     }
 
-    items.sort_by(|a, b| a.time.unwrap_or(chrono::DateTime::UNIX_EPOCH).cmp(&b.time.unwrap_or(chrono::DateTime::UNIX_EPOCH)).reverse());
+    items.sort_by(|a, b| a.time.cmp(&b.time).reverse());
 
     let mut out = String::new();
 
@@ -149,10 +150,7 @@ fn main() -> Result<()> {
     out.push_str("<ul>\n");
 
     for item in items {
-        out.push_str(&format!("<li><a href=\"{}\">[l]</a> {} {}</li>\n", item.link, item.title, match item.time{
-            Some(time) => format!("({})", time),
-            None => "".to_string(),
-        }));
+        out.push_str(&format!("<li><a href=\"{}\">[l]</a> {} {}</li>\n", item.link, item.title, item.time));
     }
 
     out.push_str("</ul>\n");