rss: publish items in chronological order

Author: Kha

zulip/integrations/rss/rss-bot

@@ -250,8 +250,13 @@ for feed_url in feed_urls:
 
     new_hashes = []  # type: List[str]
     data = feedparser.parse(feed_url)  # type: feedparser.parse
+    entries = data.entries
+    if not old_feed_hashes:
+        # On a first run, pick up the 3 most recent entries. An RSS feed has
+        # entries in reverse chronological order.
+        entries = entries[:3]
 
-    for entry in data.entries:
+    for entry in reversed(entries):
         entry_hash = compute_entry_hash(entry)  # type: str
         # An entry has either been published or updated.
         entry_time = entry.get(
@@ -265,12 +270,8 @@ for feed_url in feed_urls:
             # entries older than some threshold.
             continue
         if entry_hash in old_feed_hashes:
-            # We've already seen this. No need to process any older entries.
-            break
-        if (not old_feed_hashes) and (len(new_hashes) >= 3):
-            # On a first run, pick up the 3 most recent entries. An RSS feed has
-            # entries in reverse chronological order.
-            break
+            # We've already seen this.
+            continue
 
         feed_name = data.feed.title or feed_url  # type: str