Update changelog parser

lib/components/screens/changelog/widgets/changelog_widget.dart

@@ -22,10 +22,9 @@ class ChangelogWidget extends StatelessWidget {
   Widget _buildHeaderText() {
     return Column(
       children: <Widget>[
-        for (String headerLine in data.headerLines) ...<Widget>[
+        for (String headerLine in data.headerLines)
           Text(headerLine, textAlign: TextAlign.start),
-          const SizedBox(height: 12),
-        ],
+        const SizedBox(height: 12)
       ],
     );
   }
@@ -34,7 +33,11 @@ class ChangelogWidget extends StatelessWidget {
     return Row(
       crossAxisAlignment: CrossAxisAlignment.start,
       children: <Widget>[
-        icon,
+        Padding(
+          // move the icon down a bit to make up for the text's line height
+          padding: const EdgeInsets.only(top: 2),
+          child: icon,
+        ),
         const SizedBox(width: 24),
         Expanded(
           child: Column(

lib/misc/changelog_parser.dart

@@ -5,7 +5,7 @@ import 'package:harpy/harpy.dart';
 
 /// Parses the [ChangelogData] from a changelog file located in
 /// `android/fastlane/metadata/android/$flavor/en-US/changelogs/$versionCode.txt`.
-class ChangelogParser {
+class ChangelogParser with HarpyLogger {
   static const String linePrefix = '· ';
 
   /// Returns the [ChangelogData] for the current version.
@@ -35,10 +35,15 @@ class ChangelogParser {
       }
 
       final String headerString = changelogString.substring(0, entryStart);
+      final List<String> headerLines = _cleanEmptyLines(
+        headerString.split('\n'),
+      );
+
+      for (int i = 0; i < headerLines.length; i++) {
+        final String line = headerLines[i].trim();
 
-      for (String line in headerString.split('\n')) {
-        line = line.trim();
-        if (line.isNotEmpty) {
+        // only add empty lines if they are not the first or last line
+        if (line.isNotEmpty || i != 0 || i != headerLines.length - 1) {
           data.headerLines.add(line);
         }
       }
@@ -74,8 +79,10 @@ class ChangelogParser {
       }
 
       return data;
-    } catch (e) {
-      // changelog file not found
+    } on FlutterError {
+      // ignore asset not found exception (version has no changelog)
+    } catch (e, st) {
+      log.severe('error parsing changelog data', e, st);
       return null;
     }
   }
@@ -90,6 +97,29 @@ class ChangelogParser {
   ChangelogEntry _parseLine(String line) {
     return ChangelogEntry(line: line.trim());
   }
+
+  List<String> _cleanEmptyLines(List<String> lines) {
+    lines = _trimLeadingLines(lines);
+    lines = _trimLeadingLines(lines.reversed.toList());
+    return lines.reversed.toList();
+  }
+
+  List<String> _trimLeadingLines(List<String> lines) {
+    final List<String> cleaned = <String>[];
+
+    bool ignore = true;
+    for (String line in lines) {
+      if (ignore && line.trim().isEmpty) {
+        continue;
+      }
+
+      ignore = false;
+
+      cleaned.add(line.trim());
+    }
+
+    return cleaned;
+  }
 }
 
 /// Represents the data of a changelog for one version.