fix: Handle exceptions on APIs in a more legible way

This commit is contained in:
Alberto Ponces 2022-09-12 09:18:03 +01:00
parent e22b643375
commit 7e8ccdff43
2 changed files with 5 additions and 8 deletions

View File

@ -42,10 +42,9 @@ class GithubAPI {
); );
return response.data; return response.data;
} on Exception { } on Exception {
// ignore
}
return null; return null;
} }
}
Future<List<String>> getCommits( Future<List<String>> getCommits(
String packageName, String packageName,
@ -70,10 +69,9 @@ class GithubAPI {
(commit['commit']['message'] as String).split('\n')[0]) (commit['commit']['message'] as String).split('\n')[0])
.toList(); .toList();
} on Exception { } on Exception {
// ignore
}
return List.empty(); return List.empty();
} }
}
Future<File?> getLatestReleaseFile(String extension, String repoName) async { Future<File?> getLatestReleaseFile(String extension, String repoName) async {
try { try {
@ -104,7 +102,7 @@ class GithubAPI {
patches = list.map((patch) => Patch.fromJson(patch)).toList(); patches = list.map((patch) => Patch.fromJson(patch)).toList();
} }
} on Exception { } on Exception {
// ignore return List.empty();
} }
return patches; return patches;
} }

View File

@ -38,7 +38,7 @@ class RevancedAPI {
contributors[name] = repo['contributors']; contributors[name] = repo['contributors'];
} }
} on Exception { } on Exception {
// ignore return {};
} }
return contributors; return contributors;
} }
@ -49,10 +49,9 @@ class RevancedAPI {
List<dynamic> patches = response.data; List<dynamic> patches = response.data;
return patches.map((patch) => Patch.fromJson(patch)).toList(); return patches.map((patch) => Patch.fromJson(patch)).toList();
} on Exception { } on Exception {
// ignore
}
return List.empty(); return List.empty();
} }
}
Future<Map<String, dynamic>?> _getLatestRelease( Future<Map<String, dynamic>?> _getLatestRelease(
String extension, String extension,