Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(analytics): other iOS GA updates #652

Merged
merged 8 commits into from
Jan 15, 2025

Merge branch 'main' into mth-other-ios-ga-updates

358227a
Select commit
Loading
Failed to load commit list.
Merged

feat(analytics): other iOS GA updates #652

Merge branch 'main' into mth-other-ios-ga-updates
358227a
Select commit
Loading
Failed to load commit list.
Xcode Cloud / MBTA Transit Staging | PR Branch Workflow | iosAppRetries - iOS succeeded Jan 15, 2025 in 29m 14s

62 warnings

Report Summary
Errors 0
Test Failures 0
Analysis Issues 0
Warnings 62

Details

62 Warnings

Details

  iosApp/iosAppTests/ViewModels/StopDetailsViewModelTests.swift:25
      Using '_' to ignore the result of a Void-returning function is redundant
  iosApp/iosApp/ViewModels/SettingsViewModel.swift:108
      Line Length Violation: Line should be 120 characters or less; currently it has 123 characters (line_length)
  iosApp/iosApp/Pages/More/MoreSection.swift:38
      Line Length Violation: Line should be 120 characters or less; currently it has 122 characters (line_length)
  iosApp/iosApp/Pages/StopDetails/TripHeaderCard.swift:150
      Line Length Violation: Line should be 120 characters or less; currently it has 128 characters (line_length)
  iosApp/iosApp/Pages/StopDetails/TripHeaderCard.swift:143
      Line Length Violation: Line should be 120 characters or less; currently it has 143 characters (line_length)
  iosApp/iosApp/ViewModels/SearchViewModel.swift:127
      Redundant Optional Initialization Violation: Initializing an optional variable with nil is redundant (redundant_optional_initialization)
  iosApp/iosApp/Pages/NearbyTransit/DestinationRowView.swift:69
      Multiple Closures with Trailing Closure Violation: Trailing closure syntax should not be used when passing more than one closure argument (multiple_closures_with_trailing_closure)
  iosApp/iosApp/Analytics/DestinationRowAnalytics.swift:24
      Function Parameter Count Violation: Function should have 5 parameters or less: it currently has 6 (function_parameter_count)
  iosApp/iosApp/Pages/More/MoreLink.swift:14
      Redundant Optional Initialization Violation: Initializing an optional variable with nil is redundant (redundant_optional_initialization)
  iosApp/iosApp/Pages/Map/HomeMapViewHandlerExtension.swift:296
      Todo Violation: TODOs should be resolved (figure out something to do if ...) (todo)
  iosApp/iosApp/Pages/StopDetails/ExplainerPage.swift:118
      Line Length Violation: Line should be 120 characters or less; currently it has 132 characters (line_length)
  iosApp/iosApp/Pages/StopDetails/TripHeaderCard.swift:179
      Void Function in Ternary Violation: Using ternary to call Void functions should be avoided (void_function_in_ternary)
  iosApp/iosApp/ComponentViews/SheetNavigationLink.swift:21
      Multiple Closures with Trailing Closure Violation: Trailing closure syntax should not be used when passing more than one closure argument (multiple_closures_with_trailing_closure)
  iosApp/iosAppTests/ViewModels/StopDetailsViewModelTests.swift:59
      Instance method 'wait' is unavailable from asynchronous contexts; Use await fulfillment(of:timeout:enforceOrder:) instead; this is an error in Swift 6
  iosApp/iosAppTests/Pages/NearbyTransit/RouteCardTests.swift:22
      Multiple Closures with Trailing Closure Violation: Trailing closure syntax should not be used when passing more than one closure argument (multiple_closures_with_trailing_closure)
  iosApp/iosApp/Pages/Onboarding/OnboardingScreenView.swift:144
      Multiple Closures with Trailing Closure Violation: Trailing closure syntax should not be used when passing more than one closure argument (multiple_closures_with_trailing_closure)
  iosApp/iosApp/Pages/Onboarding/OnboardingScreenView.swift:148
      Multiple Closures with Trailing Closure Violation: Trailing closure syntax should not be used when passing more than one closure argument (multiple_closures_with_trailing_closure)
  iosApp/iosApp/ComponentViews/LocationAuthButton.swift:85
      Line Length Violation: Line should be 120 characters or less; currently it has 137 characters (line_length)
  iosApp/iosApp/ViewModels/SettingsViewModel.swift:66
      Line Length Violation: Line should be 120 characters or less; currently it has 150 characters (line_length)
  iosApp/iosApp/ViewModels/SettingsViewModel.swift:92
      Line Length Violation: Line should be 120 characters or less; currently it has 137 characters (line_length)
  iosApp/iosApp/ContentView.swift:373
      Large Tuple Violation: Tuples should have at most 2 members (large_tuple)
  iosApp/iosApp/Utils/RouteTypeExtension.swift:23
      Void Function in Ternary Violation: Using ternary to call Void functions should be avoided (void_function_in_ternary)
  iosApp/iosApp/ComponentViews/ErrorBanner.swift:56
      Line Length Violation: Line should be 120 characters or less; currently it has 126 characters (line_length)
  iosApp/iosApp/Pages/Onboarding/OnboardingScreenView.swift:134
      Line Length Violation: Line should be 120 characters or less; currently it has 124 characters (line_length)
  iosApp/iosApp/Utils/RouteTypeExtension.swift:18
      Void Function in Ternary Violation: Using ternary to call Void functions should be avoided (void_function_in_ternary)
  iosApp/iosApp/Analytics/DestinationRowAnalytics.swift:13
      Function Parameter Count Violation: Function should have 5 parameters or less: it currently has 6 (function_parameter_count)
  iosApp/iosApp/Pages/Map/MapHttpInterceptor.swift:31
      Immutable value 'error' was never used; consider replacing with '_' or removing it
  iosApp/iosApp/ComponentViews/ActionButton.swift:78
      Multiple Closures with Trailing Closure Violation: Trailing closure syntax should not be used when passing more than one closure argument (multiple_closures_with_trailing_closure)
  iosApp/iosApp/Pages/StopDetails/TripStops.swift:138
      Line Length Violation: Line should be 120 characters or less; currently it has 125 characters (line_length)
  iosApp/iosApp/Pages/NearbyTransit/NearbyTransitView.swift:17
      Type Body Length Violation: Type body should span 250 lines or less excluding comments and whitespace: currently spans 315 lines (type_body_length)
  iosApp/iosApp/Pages/NearbyTransit/DestinationRowView.swift:95
      Multiple Closures with Trailing Closure Violation: Trailing closure syntax should not be used when passing more than one closure argument (multiple_closures_with_trailing_closure)
  iosApp/iosAppTests/Views/OptionalNavigationLinkTests.swift:41
      Multiple Closures with Trailing Closure Violation: Trailing closure syntax should not be used when passing more than one closure argument (multiple_closures_with_trailing_closure)
  iosApp/iosApp/ViewModels/ErrorBannerViewModel.swift:17
      Redundant Optional Initialization Violation: Initializing an optional variable with nil is redundant (redundant_optional_initialization)
  iosApp/iosApp/Pages/StopDetails/TripStopRow.swift:162
      Line Length Violation: Line should be 120 characters or less; currently it has 121 characters (line_length)
  iosApp/iosApp/ComponentViews/UpcomingTripView.swift:150
      Line Length Violation: Line should be 120 characters or less; currently it has 143 characters (line_length)
  iosApp/iosApp/Pages/Onboarding/OnboardingScreenView.swift:81
      Line Length Violation: Line should be 120 characters or less; currently it has 139 characters (line_length)
  iosApp/iosApp/Pages/AlertDetails/AlertDetails.swift:31
      Todo Violation: TODOs should be resolved (Add all possible alert effects...) (todo)
  iosApp/iosApp/Pages/Map/MapLayerManager.swift:73
      Line Length Violation: Line should be 120 characters or less; currently it has 123 characters (line_length)
  iosApp/iosApp/Pages/Map/AnnotatedMap.swift:35
      'MapContentGestureContext' is deprecated: renamed to 'InteractionContext'
  iosApp/iosApp/Pages/LegacyStopDetails/StopDetailsFilteredRouteView.swift:230
      Multiple Closures with Trailing Closure Violation: Trailing closure syntax should not be used when passing more than one closure argument (multiple_closures_with_trailing_closure)
  iosApp/iosApp/Pages/TripDetails/TripDetailsPage.swift:14
      Type Body Length Violation: Type body should span 250 lines or less excluding comments and whitespace: currently spans 308 lines (type_body_length)
  iosApp/iosAppTests/Pages/Map/MapViewModelTests.swift:20
      Initialization of immutable value 'mapVM' was never used; consider replacing with assignment to '_' or removing it
  iosApp/iosApp/Pages/Map/HomeMapViewHandlerExtension.swift:166
      Immutable value 'tripFilter' was never used; consider replacing with '_' or removing it
  iosApp/iosApp/ContentView.swift:410
      File Length Violation: File should contain 400 lines or less: currently contains 410 (file_length)
  iosApp/iosApp/Pages/StopDetails/ExplainerPage.swift:107
      Line Length Violation: Line should be 120 characters or less; currently it has 180 characters (line_length)
  iosApp/iosApp/Utils/UIWindowExtension.swift:16
      Prefer For-Where Violation: `where` clauses are preferred over a single `if` inside a `for` (for_where)
  iosApp/iosApp/Pages/StopDetails/StopDetailsNoTripCard.swift:57
      Line Length Violation: Line should be 120 characters or less; currently it has 132 characters (line_length)
  iosApp/iosApp/Pages/NearbyTransit/NearbyTransitView.swift:564
      File Length Violation: File should contain 400 lines or less: currently contains 564 (file_length)
  iosApp/iosAppTests/ViewModels/ErrorBannerViewModelTests.swift:24
      Instance method 'wait' is unavailable from asynchronous contexts; Use await fulfillment(of:timeout:enforceOrder:) instead; this is an error in Swift 6
  iosApp/iosApp/Pages/StopDetails/TripDetailsView.swift:115
      Function Parameter Count Violation: Function should have 5 parameters or less: it currently has 6 (function_parameter_count)
  iosApp/iosApp/Pages/Map/HomeMapViewHandlerExtension.swift:257
      'MapContentGestureContext' is deprecated: renamed to 'InteractionContext'
  iosApp/iosApp/ViewModels/StopDetailsViewModel.swift:498
      File Length Violation: File should contain 400 lines or less: currently contains 498 (file_length)
  iosApp/iosApp/Utils/RouteTypeExtension.swift:21
      Void Function in Ternary Violation: Using ternary to call Void functions should be avoided (void_function_in_ternary)
  iosApp/iosApp/ProductionAppView.swift:84
      Expression implicitly coerced from 'URLResponse?' to 'Any'
  iosApp/iosAppUITests/HomeMapViewUITests.swift:79
      Initialization of immutable value 'springboard' was never used; consider replacing with assignment to '_' or removing it
  iosApp/iosApp/Pages/StopDetails/TripStopRow.swift:167
      Line Length Violation: Line should be 120 characters or less; currently it has 131 characters (line_length)
  iosApp/iosApp/ViewModels/SettingsViewModel.swift:84
      Line Length Violation: Line should be 120 characters or less; currently it has 124 characters (line_length)
  iosApp/iosApp/Pages/LegacyStopDetails/StopDetailsFilterPills.swift:81
      Multiple Closures with Trailing Closure Violation: Trailing closure syntax should not be used when passing more than one closure argument (multiple_closures_with_trailing_closure)
  iosApp/iosApp/Pages/StopDetails/TripStopRow.swift:157
      Line Length Violation: Line should be 120 characters or less; currently it has 128 characters (line_length)
  iosApp/iosApp/Utils/FetchApi.swift:35
      Immutable value 'error' was never used; consider replacing with '_' or removing it
  iosApp/iosApp/Fetchers/ViewportProvider.swift:150
      Unused Optional Binding Violation: Prefer `!= nil` over `let _ =` (unused_optional_binding)
  iosApp/iosApp/Pages/StopDetails/ExplainerPage.swift:139
      Line Length Violation: Line should be 120 characters or less; currently it has 175 characters (line_length)

Annotations

Check notice on line 257 in iosApp/iosApp/Pages/Map/HomeMapViewHandlerExtension.swift

See this annotation in the file changed.

@xcode-cloud xcode-cloud / MBTA Transit Staging | PR Branch Workflow | iosAppRetries - iOS

iosApp/iosApp/Pages/Map/HomeMapViewHandlerExtension.swift#L257

'MapContentGestureContext' is deprecated: renamed to 'InteractionContext'

Check notice on line 84 in iosApp/iosApp/ProductionAppView.swift

See this annotation in the file changed.

@xcode-cloud xcode-cloud / MBTA Transit Staging | PR Branch Workflow | iosAppRetries - iOS

iosApp/iosApp/ProductionAppView.swift#L84

Expression implicitly coerced from 'URLResponse?' to 'Any'

Check notice on line 127 in iosApp/iosApp/ViewModels/SearchViewModel.swift

See this annotation in the file changed.

@xcode-cloud xcode-cloud / MBTA Transit Staging | PR Branch Workflow | iosAppRetries - iOS

iosApp/iosApp/ViewModels/SearchViewModel.swift#L127

Redundant Optional Initialization Violation: Initializing an optional variable with nil is redundant (redundant_optional_initialization)

Check notice on line 296 in iosApp/iosApp/Pages/Map/HomeMapViewHandlerExtension.swift

See this annotation in the file changed.

@xcode-cloud xcode-cloud / MBTA Transit Staging | PR Branch Workflow | iosAppRetries - iOS

iosApp/iosApp/Pages/Map/HomeMapViewHandlerExtension.swift#L296

Todo Violation: TODOs should be resolved (figure out something to do if ...) (todo)

Check notice on line 373 in iosApp/iosApp/ContentView.swift

See this annotation in the file changed.

@xcode-cloud xcode-cloud / MBTA Transit Staging | PR Branch Workflow | iosAppRetries - iOS

iosApp/iosApp/ContentView.swift#L373

Large Tuple Violation: Tuples should have at most 2 members (large_tuple)

Check notice on line 230 in iosApp/iosApp/Pages/LegacyStopDetails/StopDetailsFilteredRouteView.swift

See this annotation in the file changed.

@xcode-cloud xcode-cloud / MBTA Transit Staging | PR Branch Workflow | iosAppRetries - iOS

iosApp/iosApp/Pages/LegacyStopDetails/StopDetailsFilteredRouteView.swift#L230

Multiple Closures with Trailing Closure Violation: Trailing closure syntax should not be used when passing more than one closure argument (multiple_closures_with_trailing_closure)

Check notice on line 166 in iosApp/iosApp/Pages/Map/HomeMapViewHandlerExtension.swift

See this annotation in the file changed.

@xcode-cloud xcode-cloud / MBTA Transit Staging | PR Branch Workflow | iosAppRetries - iOS

iosApp/iosApp/Pages/Map/HomeMapViewHandlerExtension.swift#L166

Immutable value 'tripFilter' was never used; consider replacing with '_' or removing it

Check notice on line 410 in iosApp/iosApp/ContentView.swift

See this annotation in the file changed.

@xcode-cloud xcode-cloud / MBTA Transit Staging | PR Branch Workflow | iosAppRetries - iOS

iosApp/iosApp/ContentView.swift#L410

File Length Violation: File should contain 400 lines or less: currently contains 410 (file_length)