Minor code and documentation adjustments

Author: jamesderlin

.gitignore

@@ -12,3 +12,5 @@ coverage/
 # Omit committing pubspec.lock for library packages; see
 # https://dart.dev/guides/libraries/private-files#pubspeclock.
 pubspec.lock
+
+.pubignore

Makefile

@@ -2,11 +2,11 @@ ifndef VERBOSE
 .SILENT:
 endif
 
-.PHONY: docs coverage
-
+.PHONY: docs
 docs:
 	dart doc --validate-links
 
+.PHONY: coverage
 coverage:
 	# Reference: https://pub.dev/packages/coverage
 	dart pub global run coverage:test_with_coverage \

lib/debug.dart

@@ -11,6 +11,11 @@ import 'package:stack_trace/stack_trace.dart' as stacktrace;
 /// If [packageRelative] is false, returns an absolute path.
 ///
 /// This does not work for Dart for the Web.
+///
+/// Note that [Platform.script] does not work for `import`ed files and
+/// consequently does not work with the Dart test runner.
+///
+/// [Platform.script]: https://api.dart.dev/stable/dart-io/Platform/script.html
 String currentDartFilePath({bool packageRelative = false}) {
   var caller = stacktrace.Frame.caller(1);
   return packageRelative ? caller.library : caller.uri.toFilePath();

lib/misc.dart

@@ -83,7 +83,7 @@ extension PadLeftExtension on int {
   /// Returns a string representation of this [int], left-padded with zeroes if
   /// necessary to have the specified minimum number of characters.
   ///
-  /// If this [int] is negative, the negative sign is included in number of
+  /// If this [int] is negative, the negative sign is included in the number of
   /// characters.
   ///
   /// Examples:
@@ -122,7 +122,7 @@ extension ImpliesExtension on bool {
   /// Returns whether this [bool] [logically implies][1] [consequence].
   ///
   /// Given two boolean expressions *p* and *q*, the results of "*p* implies
-  /// *q*"" (denoted as *p → q*) are shown by the following truth table:
+  /// *q*" (denoted as *p → q*) are shown by the following truth table:
   ///
   /// |  p  |  q  | p → q |
   /// |:---:|:---:|:----------:|
@@ -143,10 +143,7 @@ extension FutureCast<T> on Future<T> {
   /// Casts a `Future<T>` to a `Future<R>`.
   ///
   // Motivated by: <https://stackoverflow.com/q/72576065/>
-  Future<R> cast<R>() async {
-    var result = await this;
-    return result as R;
-  }
+  Future<R> cast<R>() async => (await this) as R;
 }
 
 /// An implementation of [Future] that allows synchronously retrieving the

lib/readable_numbers.dart

@@ -4,6 +4,7 @@ library;
 import 'dart:math' as math;
 
 import 'math.dart';
+import 'misc.dart';
 
 const _siMacroPrefixes = ['', 'K', 'M', 'G', 'T', 'P', 'E', 'Z', 'Y'];
 const _siMicroPrefixes = ['m', '\u03BC', 'n', 'p', 'f', 'a', 'z', 'y'];
@@ -158,8 +159,9 @@ String readableDuration(Duration duration, {int? precision}) {
 
   var secondsString = '';
   if (microseconds > 0) {
+    // Strip off trailing zeroes from the fractional portion.
     var fractionalSecondsString =
-        '$microseconds'.padLeft(6, '0').replaceAll(RegExp(r'0+$'), '');
+        microseconds.padLeft(6).replaceAll(RegExp(r'0+$'), '');
     secondsString = '$seconds.${fractionalSecondsString}s';
   } else if (seconds > 0) {
     secondsString = '${seconds}s';

pubspec.yaml

@@ -2,7 +2,7 @@ name: dartbag
 
 # Description must be 60-180 characters to appease pub.dev's scoring system.
 description: A grab-bag of miscellaneous, lightweight utility libraries for Dart.
-version: 0.7.0+1
+version: 0.7.1-dev
 repository: https://github.com/jamesderlin/dartbag
 
 environment: