Skip to content
This repository has been archived by the owner on Feb 19, 2020. It is now read-only.

Commit

Permalink
Fix new lints from pedantic and be more strict about checking them (#30)
Browse files Browse the repository at this point in the history
  • Loading branch information
kevmoo committed Jan 23, 2019
1 parent 3478638 commit 42b293f
Show file tree
Hide file tree
Showing 6 changed files with 7 additions and 7 deletions.
2 changes: 1 addition & 1 deletion .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ dart:
dart_task:
- test
- dartfmt
- dartanalyzer
- dartanalyzer: --fatal-infos --fatal-warnings .

# Only building master means that we don't run two builds for each pull request.
branches:
Expand Down
2 changes: 1 addition & 1 deletion example/example.dart
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ main(List<String> args) {
var port = int.parse(results['port'] as String);
var standalone = results['standalone'] as bool;

if (results.rest.length > 0) {
if (results.rest.isNotEmpty) {
print('Got unexpected arguments: "${results.rest.join(' ')}".\n\nUsage:\n');
print(parser.usage);
exit(1);
Expand Down
2 changes: 1 addition & 1 deletion example/src/examples/cow_repository.dart
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@ class CopyAndWriteRepository extends PackageRepository {
.where((pv) => pv.versionString == version)
.toList()
.then((List<PackageVersion> versions) {
if (versions.length >= 1) return versions.first;
if (versions.isNotEmpty) return versions.first;
return null;
});
}
Expand Down
2 changes: 1 addition & 1 deletion example/src/examples/file_repository.dart
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ class FileRepository extends PackageRepository {
.where((pv) => pv.versionString == version)
.toList()
.then((List<PackageVersion> versions) {
if (versions.length >= 1) return versions.first;
if (versions.isNotEmpty) return versions.first;
return null;
});
}
Expand Down
2 changes: 1 addition & 1 deletion example/src/examples/http_proxy_repository.dart
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ class HttpProxyRepository extends PackageRepository {
.where((v) => v.packageName == package && v.versionString == version)
.toList()
.then((List<PackageVersion> versions) {
if (versions.length >= 1) return versions.first;
if (versions.isNotEmpty) return versions.first;
return null;
});
}
Expand Down
4 changes: 2 additions & 2 deletions lib/shelf_pubserver.dart
Original file line number Diff line number Diff line change
Expand Up @@ -240,7 +240,7 @@ class ShelfPubServer {
}

var packageVersions = await repository.versions(package).toList();
if (packageVersions.length == 0) {
if (packageVersions.isEmpty) {
return new shelf.Response.notFound(null);
}

Expand Down Expand Up @@ -416,7 +416,7 @@ class ShelfPubServer {

Future<shelf.Response> _addUploader(String package, String body) async {
var parts = body.split('=');
if (parts.length == 2 && parts[0] == 'email' && parts[1].length > 0) {
if (parts.length == 2 && parts[0] == 'email' && parts[1].isNotEmpty) {
try {
var user = Uri.decodeQueryComponent(parts[1]);
await repository.addUploader(package, user);
Expand Down

0 comments on commit 42b293f

Please sign in to comment.