chore(serverpod_logging): Lower sdk constraint to 3.6, enable workspace, tweak ci, stop tracking pubspec.lock, format code (#119)

- lower the SDK floor to ^3.6.0, 
- pull the packages into the workspace, 
- stop tracking the root pubspec.lock, 
- reformat to 3.6 short style, 
- and tweak the CI matrix.

Author: nielsenko

.github/workflows/ci.yml

@@ -42,18 +42,13 @@ jobs:
         dart: [3.6, 3.12]
         package: [cli_tools, config, serverpod_logging]
         platform: [ubuntu-latest]
-        exclude:
-          - package: serverpod_logging
         include:
           - package: cli_tools
             platform: windows-latest
             dart: 3.6
           - package: cli_tools
             platform: macos-latest
             dart: 3.6
-          - package: serverpod_logging
-            dart: 3.10.3
-            platform: ubuntu-latest
     runs-on: ${{ matrix.platform }}
     defaults:
       run:

.gitignore

@@ -2,3 +2,6 @@
 
 .idea/
 *.iml
+
+# Workspace of library packages; the root lockfile is regenerated by CI.
+pubspec.lock

packages/serverpod_logging/lib/src/log.dart

@@ -71,39 +71,39 @@ class Log {
 extension LogConvenience on Log {
   /// Logs [message] at [LogLevel.debug].
   void debug(String message, {Map<String, Object?>? metadata}) => this(
-    LogLevel.debug,
-    () => LogEntry(
-      time: DateTime.now(),
-      level: LogLevel.debug,
-      message: message,
-      scope: currentScope,
-      metadata: metadata,
-    ),
-  );
+        LogLevel.debug,
+        () => LogEntry(
+          time: DateTime.now(),
+          level: LogLevel.debug,
+          message: message,
+          scope: currentScope,
+          metadata: metadata,
+        ),
+      );
 
   /// Logs [message] at [LogLevel.info].
   void info(String message, {Map<String, Object?>? metadata}) => this(
-    LogLevel.info,
-    () => LogEntry(
-      time: DateTime.now(),
-      level: LogLevel.info,
-      message: message,
-      scope: currentScope,
-      metadata: metadata,
-    ),
-  );
+        LogLevel.info,
+        () => LogEntry(
+          time: DateTime.now(),
+          level: LogLevel.info,
+          message: message,
+          scope: currentScope,
+          metadata: metadata,
+        ),
+      );
 
   /// Logs [message] at [LogLevel.warning].
   void warning(String message, {Map<String, Object?>? metadata}) => this(
-    LogLevel.warning,
-    () => LogEntry(
-      time: DateTime.now(),
-      level: LogLevel.warning,
-      message: message,
-      scope: currentScope,
-      metadata: metadata,
-    ),
-  );
+        LogLevel.warning,
+        () => LogEntry(
+          time: DateTime.now(),
+          level: LogLevel.warning,
+          message: message,
+          scope: currentScope,
+          metadata: metadata,
+        ),
+      );
 
   /// Logs [message] at [LogLevel.error], optionally attaching an [error]
   /// value and [stackTrace].
@@ -112,18 +112,19 @@ extension LogConvenience on Log {
     Object? error,
     StackTrace? stackTrace,
     Map<String, Object?>? metadata,
-  }) => this(
-    LogLevel.error,
-    () => LogEntry(
-      time: DateTime.now(),
-      level: LogLevel.error,
-      message: message,
-      scope: currentScope,
-      error: error,
-      stackTrace: stackTrace,
-      metadata: metadata,
-    ),
-  );
+  }) =>
+      this(
+        LogLevel.error,
+        () => LogEntry(
+          time: DateTime.now(),
+          level: LogLevel.error,
+          message: message,
+          scope: currentScope,
+          error: error,
+          stackTrace: stackTrace,
+          metadata: metadata,
+        ),
+      );
 
   /// Whether debug-level messages are currently forwarded to the writer.
   /// Use to gate expensive message construction:

packages/serverpod_logging/lib/src/log_types.dart

@@ -58,13 +58,14 @@ class LogScope {
     required String id,
     required String label,
     Map<String, Object?>? metadata,
-  }) => LogScope(
-    id: id,
-    label: label,
-    startTime: DateTime.now(),
-    parent: this,
-    metadata: metadata,
-  );
+  }) =>
+      LogScope(
+        id: id,
+        label: label,
+        startTime: DateTime.now(),
+        parent: this,
+        metadata: metadata,
+      );
 }
 
 /// A single log entry. Always belongs to a [LogScope].
@@ -162,17 +163,18 @@ class MultiLogWriter extends LogWriter {
     required Duration duration,
     Object? error,
     StackTrace? stackTrace,
-  }) => _writers
-      .map(
-        (w) => w.closeScope(
-          scope,
-          success: success,
-          duration: duration,
-          error: error,
-          stackTrace: stackTrace,
-        ),
-      )
-      .wait;
+  }) =>
+      _writers
+          .map(
+            (w) => w.closeScope(
+              scope,
+              success: success,
+              duration: duration,
+              error: error,
+              stackTrace: stackTrace,
+            ),
+          )
+          .wait;
 
   @override
   Future<void> close() => _writers.map((w) => w.close()).wait;

packages/serverpod_logging/pubspec.yaml

@@ -1,14 +1,16 @@
 name: serverpod_logging
 description: Shared serverpod logging utilities.
-version: 0.0.1
+version: 0.1.0
 repository: https://github.com/serverpod/serverpod
 homepage: https://serverpod.dev
 issue_tracker: https://github.com/serverpod/serverpod/issues
 
 environment:
-  sdk: ^3.10.3
+  sdk: ^3.6.0
 
-dependencies: 
+resolution: workspace
+
+dependencies:
   meta: ^1.15.0
 
 dev_dependencies:

packages/serverpod_logging/test/log_progress_test.dart

@@ -11,15 +11,17 @@ void main() {
       log = Log(writer);
     });
 
-    test('when the runner returns true, '
+    test(
+        'when the runner returns true, '
         'then the scope closes with success true', () async {
       await log.progress('op', () async => true);
 
       expect(writer.closedScopes, hasLength(1));
       expect(writer.closedScopes.first.success, isTrue);
     });
 
-    test('when the runner returns false, '
+    test(
+        'when the runner returns false, '
         'then the scope closes with success false', () async {
       await log.progress('op', () async => false);
 
@@ -37,14 +39,16 @@ void main() {
       log = Log(writer);
     });
 
-    test('when no isSuccess is provided, '
+    test(
+        'when no isSuccess is provided, '
         'then the scope closes with success true', () async {
       await log.progress<String>('op', () async => 'ok');
 
       expect(writer.closedScopes.first.success, isTrue);
     });
 
-    test('when isSuccess is provided, '
+    test(
+        'when isSuccess is provided, '
         'then the scope uses its verdict', () async {
       await log.progress<String>(
         'op',
@@ -65,7 +69,8 @@ void main() {
       log = Log(writer);
     });
 
-    test('when the runner throws, '
+    test(
+        'when the runner throws, '
         'then the scope closes with success false and rethrows', () async {
       await expectLater(
         log.progress<bool>('op', () async => throw StateError('boom')),