Merge branch 'main' into feat/info-screen

.github/workflows/main.yaml

@@ -14,3 +14,4 @@ jobs:
       flutter_version: 2.10.5
       coverage_excludes: "lib/gen/*.dart"
       test_optimization: false
+      

.github/workflows/spell_check.yaml

@@ -0,0 +1,21 @@
+name: spell_check
+
+concurrency:
+  group: ${{ github.workflow }}-${{ github.ref }}
+  cancel-in-progress: true
+  
+on: [pull_request]
+
+jobs:
+  build:
+    name: Spell Check
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v2
+
+      - name: Check spelling
+        uses: zwaldowski/cspell-action@v1
+        with:
+          paths: '**/*.{dart,arb}'
+          config: .vscode/cspell.json

.vscode/cspell.json

@@ -0,0 +1,41 @@
+{
+    "version": "0.2",
+    "enabled": true,
+    "language": "en",
+    "words": [
+        "animatronic",
+        "argb",
+        "audioplayers",
+        "backbox",
+        "bezier",
+        "cupertino",
+        "dashbook",
+        "deserialization",
+        "dpad",
+        "endtemplate",
+        "firestore",
+        "gapless",
+        "goldens",
+        "leaderboard",
+        "loadables",
+        "mixins",
+        "mocktail",
+        "multiball",
+        "multiballs",
+        "occluder",
+        "pixelated",
+        "pixeloid",
+        "rects",
+        "rrect",
+        "serializable",
+        "sparky's",
+        "theming",
+        "unawaited",
+        "unfocus",
+        "unlayered",
+        "vsync"
+    ],
+    "ignorePaths": [
+        ".github/workflows/**"
+    ]
+}

lib/game/behaviors/scoring_behavior.dart

@@ -15,7 +15,7 @@ import 'package:pinball_flame/pinball_flame.dart';
 /// {@endtemplate}
 class ScoringBehavior extends Component
     with HasGameRef, FlameBlocReader<GameBloc, GameState> {
-  /// {@macto scoring_behavior}
+  /// {@macro scoring_behavior}
   ScoringBehavior({
     required Points points,
     required Vector2 position,

lib/game/components/backbox/displays/initials_input_display.dart

@@ -77,7 +77,7 @@ class InitialsInputDisplay extends Component with HasGameRef {
     );
   }
 
-  /// Returns the current inputed initials
+  /// Returns the current entered initials
   String get initials => children
       .whereType<InitialsLetterPrompt>()
       .map((prompt) => prompt.char)

lib/game/pinball_game.dart

@@ -143,7 +143,7 @@ class PinballGame extends PinballForge2DGame
       final rocket = descendants().whereType<RocketSpriteComponent>().first;
       final bounds = rocket.topLeftPosition & rocket.size;
 
-      // NOTE(wolfen): As long as Flame does not have https://github.com/flame-engine/flame/issues/1586 we need to check it at the highest level manually.
+      // NOTE: As long as Flame does not have https://github.com/flame-engine/flame/issues/1586 we need to check it at the highest level manually.
       if (bounds.contains(info.eventPosition.game.toOffset())) {
         descendants().whereType<Plunger>().single.pullFor(2);
       } else {

packages/pinball_audio/lib/src/pinball_audio.dart

@@ -244,7 +244,7 @@ class PinballAudioPlayer {
     return audios.values.map((a) => a.load()).toList();
   }
 
-  /// Plays the received auido
+  /// Plays the received audio
   void play(PinballAudio audio) {
     assert(
       audios.containsKey(audio),

packages/pinball_components/lib/src/components/ball/ball.dart

@@ -68,7 +68,7 @@ class Ball extends BodyComponent with Layered, InitialPosition, ZIndex {
     return world.createBody(bodyDef)..createFixtureFromShape(shape, 1);
   }
 
-  /// Immediatly and completly [stop]s the ball.
+  /// Immediately and completely [stop]s the ball.
   ///
   /// The [Ball] will no longer be affected by any forces, including it's
   /// weight and those emitted from collisions.

packages/pinball_components/lib/src/components/bumping_behavior.dart

@@ -12,7 +12,7 @@ class BumpingBehavior extends ContactBehavior {
   /// Determines how strong the bump is.
   final double _strength;
 
-  /// This is used to recoginze the current state of a contact manifold in world
+  /// This is used to recognize the current state of a contact manifold in world
   /// coordinates.
   @visibleForTesting
   final WorldManifold worldManifold = WorldManifold();

packages/pinball_components/lib/src/components/flipper/behaviors/flipper_jointing_behavior.dart

@@ -40,7 +40,7 @@ class _FlipperAnchor extends JointAnchor {
 }
 
 /// {@template flipper_anchor_revolute_joint_def}
-/// Hinges one end of [Flipper] to a [_FlipperAnchor] to achieve a potivoting
+/// Hinges one end of [Flipper] to a [_FlipperAnchor] to achieve a pivoting
 /// motion.
 /// {@endtemplate}
 class _FlipperAnchorRevoluteJointDef extends RevoluteJointDef {

packages/pinball_components/lib/src/components/joint_anchor.dart

@@ -12,7 +12,7 @@ import 'package:pinball_components/pinball_components.dart';
 /// initialize(
 ///   dynamicBody.body,
 ///   anchor.body,
-///   dynabmicBody.body + anchor.body.position,
+///   dynamicBody.body + anchor.body.position,
 /// );
 /// ```
 /// {@endtemplate}

packages/pinball_components/lib/src/components/plunger.dart

@@ -6,7 +6,7 @@ import 'package:pinball_flame/pinball_flame.dart';
 
 /// {@template plunger}
 /// [Plunger] serves as a spring, that shoots the ball on the right side of the
-/// playfield.
+/// play field.
 ///
 /// [Plunger] ignores gravity so the player controls its downward [pull].
 /// {@endtemplate}

packages/pinball_components/lib/src/components/slingshot.dart

@@ -37,7 +37,7 @@ class Slingshot extends BodyComponent with InitialPosition {
   })  : _angle = angle,
         super(
           children: [
-            _SlinghsotSpriteComponent(spritePath, angle: angle),
+            _SlingshotSpriteComponent(spritePath, angle: angle),
             BumpingBehavior(strength: 20),
           ],
           renderBody: false,
@@ -90,8 +90,8 @@ class Slingshot extends BodyComponent with InitialPosition {
   }
 }
 
-class _SlinghsotSpriteComponent extends SpriteComponent with HasGameRef {
-  _SlinghsotSpriteComponent(
+class _SlingshotSpriteComponent extends SpriteComponent with HasGameRef {
+  _SlingshotSpriteComponent(
     String path, {
     required double angle,
   })  : _path = path,

packages/pinball_components/test/src/components/bumping_behavior_test.dart

@@ -32,7 +32,7 @@ void main() {
     });
 
     flameTester.testGameWidget(
-      'the bump is greater when the strengh is greater',
+      'the bump is greater when the strength is greater',
       setUp: (game, tester) async {
         final component1 = _TestBodyComponent();
         final behavior1 = BumpingBehavior(strength: 1)

packages/pinball_components/test/src/components/flapper/flapper_test.dart

@@ -67,7 +67,7 @@ void main() {
       },
     );
 
-    flameTester.test('adds a FlapperSpiningBehavior to FlapperEntrance',
+    flameTester.test('adds a FlapperSpinningBehavior to FlapperEntrance',
         (game) async {
       final flapper = Flapper();
       await game.ensureAdd(flapper);

packages/pinball_components/test/src/components/initial_position_test.dart

@@ -54,7 +54,7 @@ void main() {
     );
 
     flameTester.test(
-      'deafaults to zero '
+      'defaults to zero '
       'when no initialPosition is given',
       (game) async {
         final component = TestBodyComponent();

packages/pinball_components/test/src/components/plunger_test.dart

@@ -329,7 +329,7 @@ void main() {
       );
 
       flameTester.test(
-        'connected body collison enabled',
+        'connected body collision enabled',
         (game) async {
           await game.ensureAdd(plunger);
           await game.ensureAdd(anchor);

packages/pinball_flame/lib/src/canvas/canvas_component.dart

@@ -12,7 +12,7 @@ typedef PaintFunction = void Function(Paint);
 /// {@template canvas_component}
 /// Allows listening before the rendering of [Sprite]s.
 ///
-/// The existance of this class is to hack around the fact that Flame doesn't
+/// The existence of this class is to hack around the fact that Flame doesn't
 /// provide a global way to modify the default [Paint] before rendering a
 /// [Sprite].
 /// {@endtemplate}

packages/pinball_flame/lib/src/pinball_forge2d_game.dart

@@ -4,7 +4,7 @@ import 'package:flame/game.dart';
 import 'package:flame_forge2d/flame_forge2d.dart';
 import 'package:flame_forge2d/world_contact_listener.dart';
 
-// NOTE(wolfen): This should be removed when https://github.com/flame-engine/flame/pull/1597 is solved.
+// NOTE: This should be removed when https://github.com/flame-engine/flame/pull/1597 is solved.
 /// {@template pinball_forge2d_game}
 /// A [Game] that uses the Forge2D physics engine.
 /// {@endtemplate}

packages/pinball_flame/test/src/canvas/canvas_component_test.dart

@@ -50,7 +50,7 @@ void main() {
     );
 
     flameTester.testGameWidget(
-      'calls onSpritePainted when paiting a sprite',
+      'calls onSpritePainted when painting a sprite',
       setUp: (game, tester) async {
         final spriteComponent = _TestSpriteComponent();
 

packages/pinball_flame/test/src/component_controller_test.dart

@@ -38,9 +38,9 @@ void main() {
         final component = Component();
         final controller = TestComponentController(component);
 
-        final anotherComponet = Component();
+        final anotherComponent = Component();
         await expectLater(
-          () async => await anotherComponet.add(controller),
+          () async => await anotherComponent.add(controller),
           throwsAssertionError,
         );
       },

packages/pinball_flame/test/src/pinball_forge2d_game_test.dart

@@ -19,7 +19,7 @@ void main() {
     });
 
     flameTester.test(
-      'screenToFlameWorld throws UnimpelementedError',
+      'screenToFlameWorld throws UnimplementedError',
       (game) async {
         expect(
           () => game.screenToFlameWorld(Vector2.zero()),
@@ -29,7 +29,7 @@ void main() {
     );
 
     flameTester.test(
-      'screenToWorld throws UnimpelementedError',
+      'screenToWorld throws UnimplementedError',
       (game) async {
         expect(
           () => game.screenToWorld(Vector2.zero()),
@@ -39,7 +39,7 @@ void main() {
     );
 
     flameTester.test(
-      'worldToScreen throws UnimpelementedError',
+      'worldToScreen throws UnimplementedError',
       (game) async {
         expect(
           () => game.worldToScreen(Vector2.zero()),

packages/pinball_ui/lib/src/widgets/animated_ellipsis_text.dart

@@ -2,7 +2,7 @@ import 'dart:async';
 
 import 'package:flutter/material.dart';
 
-/// {@tempalte animated_ellipsis_text}
+/// {@template animated_ellipsis_text}
 /// Every 500 milliseconds, it will add a new `.` at the end of the given
 /// [text]. Once 3 `.` have been added (e.g. `Loading...`), it will reset to
 /// zero ellipsis and start over again.

packages/pinball_ui/lib/src/widgets/crt_background.dart

@@ -2,7 +2,7 @@ import 'package:flutter/material.dart';
 import 'package:pinball_ui/pinball_ui.dart';
 
 /// {@template crt_background}
-/// [BoxDecoration] that provides a CRT-like background efffect.
+/// [BoxDecoration] that provides a CRT-like background effect.
 /// {@endtemplate}
 class CrtBackground extends BoxDecoration {
   /// {@macro crt_background}

packages/pinball_ui/lib/src/widgets/pinball_dpad_button.dart

@@ -2,7 +2,7 @@ import 'package:flutter/material.dart';
 import 'package:pinball_ui/gen/gen.dart';
 import 'package:pinball_ui/pinball_ui.dart';
 
-/// Enum with all possibile directions of a [PinballDpadButton].
+/// Enum with all possible directions of a [PinballDpadButton].
 enum PinballDpadDirection {
   /// Up
   up,

packages/platform_helper/test/src/platform_helper_test.dart

@@ -27,7 +27,7 @@ void main() {
       });
 
       test(
-        'returns false when defaultTargetPlatform is niether iOS nor android',
+        'returns false when defaultTargetPlatform is neither iOS nor android',
         () async {
           debugDefaultTargetPlatformOverride = TargetPlatform.macOS;
           expect(PlatformHelper().isMobile, isFalse);

test/game/components/backbox/displays/initials_input_display_test.dart

@@ -129,7 +129,7 @@ void main() {
       },
     );
 
-    String? submitedInitials;
+    String? submittedInitials;
     flameTester.testGameWidget(
       'submits the initials',
       setUp: (game, tester) async {
@@ -138,7 +138,7 @@ void main() {
           score: 1000,
           characterIconPath: game.characterIconPath,
           onSubmit: (value) {
-            submitedInitials = value;
+            submittedInitials = value;
           },
         );
         await game.pump(component);
@@ -147,7 +147,7 @@ void main() {
         await tester.pump();
       },
       verify: (game, tester) async {
-        expect(submitedInitials, equals('AAA'));
+        expect(submittedInitials, equals('AAA'));
       },
     );
 

test/game/components/bottom_group_test.dart

@@ -95,9 +95,9 @@ void main() {
             ),
           );
 
-          final basebottomGroups =
+          final baseBottomGroups =
               bottomGroup.descendants().whereType<Baseboard>();
-          expect(basebottomGroups.length, equals(2));
+          expect(baseBottomGroups.length, equals(2));
         },
       );
 

test/game/components/game_bloc_status_listener_test.dart

@@ -162,7 +162,7 @@ void main() {
         );
 
         flameTester.test(
-          'removes FlipperKeyControllingBehavior from Fipper',
+          'removes FlipperKeyControllingBehavior from Flipper',
           (game) async {
             final component = GameBlocStatusListener();
             final repository = _MockLeaderboardRepository();
@@ -242,7 +242,7 @@ void main() {
         );
 
         flameTester.test(
-          'adds key controlling behavior to Fippers when the game is started',
+          'adds key controlling behavior to Flippers when the game is started',
           (game) async {
             final component = GameBlocStatusListener();
             final repository = _MockLeaderboardRepository();

test/select_character/cubit/character_theme_cubit_test.dart

@@ -14,7 +14,7 @@ void main() {
     });
 
     blocTest<CharacterThemeCubit, CharacterThemeState>(
-      'charcterSelected emits selected character theme',
+      'characterSelected emits selected character theme',
       build: CharacterThemeCubit.new,
       act: (bloc) => bloc.characterSelected(const SparkyTheme()),
       expect: () => [